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

Failure recover for client side #272

Closed
HarrisChu opened this issue Apr 10, 2023 · 0 comments · Fixed by #273
Closed

Failure recover for client side #272

HarrisChu opened this issue Apr 10, 2023 · 0 comments · Fixed by #273
Labels
affects/none PR/issue: this bug affects none version. process/fixed Process of bug severity/none Severity of bug type/bug Type: something is unexpected

Comments

@HarrisChu
Copy link
Contributor

  1. start client and then connect a graph address.
  2. query forever
  3. shutdown the graph service
  4. wait for 3 seconds
  5. start the graph service

expected result:
after graph restart, the client query successfully

actual result:
always raise error

  1. if the connection is invalid, it would turn back the pool, and would exceed the max connection limit
    image

  2. if the err is write: broken pipe, it would always be an error here
    image
@HarrisChu HarrisChu added the type/bug Type: something is unexpected label Apr 10, 2023
@github-actions github-actions bot added affects/none PR/issue: this bug affects none version. severity/none Severity of bug labels Apr 10, 2023
@HarrisChu HarrisChu changed the title High available for client side Failure recover for client side Apr 10, 2023
HarrisChu added a commit to HarrisChu/nebula-go that referenced this issue Apr 11, 2023
@HarrisChu
fix vesoft-inc#272
dbf6afd
@Aiee Aiee mentioned this issue Apr 12, 2023
Merged
3 tasks
@Aiee Aiee closed this as completed in #273 Apr 12, 2023
Aiee pushed a commit that referenced this issue Apr 12, 2023
@HarrisChu
fix #272 (#273)
fae48de 
* add test

* fix test case

* fix #272
@github-actions github-actions bot added the process/fixed Process of bug label Apr 12, 2023
@wey-gu wey-gu mentioned this issue Apr 15, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
affects/none PR/issue: this bug affects none version. process/fixed Process of bug severity/none Severity of bug type/bug Type: something is unexpected
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix #272 HarrisChu/nebula-go
1 participant
@HarrisChu