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

refactor client.connect to instantiate a new Channel for each connect #2513

Closed
hokiegeek2 opened this issue Jun 16, 2023 · 0 comments · Fixed by #2514
Closed

refactor client.connect to instantiate a new Channel for each connect #2513

hokiegeek2 opened this issue Jun 16, 2023 · 0 comments · Fixed by #2514
Assignees
@hokiegeek2
Labels
bug Something isn't working

Comments

@hokiegeek2
Copy link
Contributor

Describe the bug
Nightly Arkouda testing by the Chapel team is failing b/c the slurm launch method can change the server host for each connection. Consequently, need to create a new Channel instance for each connection

Is this a Blocking Issue
Preventing Arkouda nightly testing

Additional context
Add any other context about the problem here.
@hokiegeek2 hokiegeek2 added the bug Something isn't working label Jun 16, 2023
@hokiegeek2 hokiegeek2 self-assigned this Jun 16, 2023
@hokiegeek2 hokiegeek2 changed the title refactor client.connect() to instantiate a new Channel for each connection refactor client.connect to instantiate a new Channel for each connect Jun 16, 2023
hokiegeek2 added a commit to hokiegeek2/arkouda that referenced this issue Jun 16, 2023
@hokiegeek2
refactored to create a new Channel for each client.connect execution B…
439a994 
…ears-R-Us#2513
@hokiegeek2 hokiegeek2 mentioned this issue Jun 16, 2023
Merged
Ethan-DeBandi99 pushed a commit that referenced this issue Jun 20, 2023
@hokiegeek2
refactored to create a new Channel for each client.connect execution #…
f417c43 
…2513 (#2514)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@hokiegeek2 hokiegeek2

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

refactored to create a new Channel for each client.connect execution hokiegeek2/arkouda
1 participant
@hokiegeek2