Skip to content

Conversation

@ericl
Copy link
Contributor

@ericl ericl commented Sep 27, 2018

What do these changes do?

This has been superceded by the new multi-policy support.

#2736

@ericl
Copy link
Contributor Author

ericl commented Sep 27, 2018

cc @eugenevinitsky

@AmplabJenkins
Copy link

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

@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/8401/
Test PASSed.

Copy link
Contributor

@richardliaw richardliaw left a comment

Choose a reason for hiding this comment

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

Looks good to me; good to get @eugenevinitsky to comment though.

@eugenevinitsky
Copy link
Contributor

Looked it over, this does seem like everything was successfully removed.

@ericl ericl merged commit 2019b41 into ray-project:master Oct 1, 2018
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