Skip to content

Conversation

@atumanov
Copy link
Contributor

No description provided.

@AmplabJenkins
Copy link

Merged build finished. Test PASSed.

@AmplabJenkins
Copy link

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Ray-PRB/73/
Test PASSed.

}
return; /* No workers available -- terminate early. */
}
/* Terminate early if there are no more resources available. */
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This assumes that all tasks consume resources, e.g., that we can't do

@ray.remote(num_cpus=0, num_gpus=0)

which is probably fine.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yeah, that's right. I assume that a task must consume some resources, even if it's a fraction of a core.

}
}
if (!resources_available) {
return; /* No resources available -- terminate early. */
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This comment should be on the preceding line. Same with line 584.

@AmplabJenkins
Copy link

Merged build finished. Test PASSed.

@AmplabJenkins
Copy link

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Ray-PRB/74/
Test PASSed.

@pcmoritz pcmoritz merged commit 3159a78 into ray-project:master Feb 23, 2017
@pcmoritz pcmoritz deleted the terminate-dispatch-early branch February 23, 2017 08:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants