Skip to content

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

Merged
robertnishihara merged 4 commits into
ray-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 into
ray-project:masterfrom
ericl:cleanup-ar

Conversation

@ericl

@ericl ericl commented Nov 25, 2018

Copy link
Copy Markdown
Contributor

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

ericl commented Nov 27, 2018

Copy link
Copy Markdown
Contributor Author

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