Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

add logic to call exit(0) in DefaultServerDaemon when 2..n ServerDaemons configured #1719

Closed
hokiegeek2 opened this issue Aug 25, 2022 · 2 comments
Closed

add logic to call exit(0) in DefaultServerDaemon when 2..n ServerDaemons configured #1719

hokiegeek2 opened this issue Aug 25, 2022 · 2 comments

Comments

@hokiegeek2
Copy link
Contributor

As @ronawho reported on 8/19, ServerDaemon.shutdown() fails on certain Cray machines due to a Chapel bug. Need to prevent invocation of the exit(0) function in the DefaultServerDaemon unless there are 2..n ServerDaemons configured; this update will fix the default arkouda_server configuration while a fix for the root cause will be done in another, follow-up issue.
@hokiegeek2 hokiegeek2 changed the title add logic to only call exit(0) in DefaultServerDaemon when 2..n ServerDaemons configured add logic to call exit(0) in DefaultServerDaemon when 2..n ServerDaemons configured Aug 25, 2022
@hokiegeek2
Copy link
Contributor Author

hokiegeek2 commented Aug 25, 2022 •
edited
Loading

Logic is working: exit(0) not called in default arkouda_server config...

image

...but is called if there are 2..n ArkoudaServerDaemons configured:

image

hokiegeek2 added a commit to hokiegeek2/arkouda that referenced this issue Aug 25, 2022
@hokiegeek2
added logic to call exit(0) only when there are 2..n ArkoudaServerDae…
c4def5c 
…mons Bears-R-Us#1719
Ethan-DeBandi99 pushed a commit that referenced this issue Aug 25, 2022
@hokiegeek2
added logic to call exit(0) only when there are 2..n ArkoudaServerDae…
cd6b91a 
…mons #1719 (#1720)
@Ethan-DeBandi99
Copy link
Contributor

Corresponding PR #1720 closed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@hokiegeek2 @Ethan-DeBandi99