Skip to content

Don't put entire actor registry in debug string since it's too long#3395

Merged
robertnishihara merged 4 commits intoray-project:masterfrom
ericl:cleanup-ar
Nov 28, 2018
Merged

Don't put entire actor registry in debug string since it's too long#3395
robertnishihara merged 4 commits intoray-project:masterfrom
ericl:cleanup-ar

Conversation

@ericl
Copy link
Copy Markdown
Contributor

@ericl ericl commented Nov 25, 2018

No description provided.

@AmplabJenkins
Copy link
Copy Markdown

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

@ericl
Copy link
Copy Markdown
Contributor Author

ericl commented Nov 27, 2018

Ping

@robertnishihara
Copy link
Copy Markdown
Collaborator

@ericl Compilation error

In file included from /Users/travis/build/ray-project/ray/src/ray/raylet/actor_registration.cc:1:
/Users/travis/build/ray-project/ray/src/ray/raylet/actor_registration.h:107:8: error: private field 'alive_' is not used [-Werror,-Wunused-private-field]
  bool alive_;
       ^
1 error generated.
make[2]: *** [src/ray/CMakeFiles/ray_objlib.dir/raylet/actor_registration.cc.o] Error 1
make[1]: *** [src/ray/CMakeFiles/ray_objlib.dir/all] Error 2
make[1]: *** Waiting for unfinished jobs....
3 warnings generated.

@robertnishihara robertnishihara merged commit c2108ca into ray-project:master Nov 28, 2018
@robertnishihara robertnishihara deleted the cleanup-ar branch November 28, 2018 00:48
@AmplabJenkins
Copy link
Copy Markdown

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

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.

3 participants