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.

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

keep opensergoclient alive #7

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

jnan806
Copy link
Collaborator

@jnan806 jnan806 commented Oct 10, 2022

To keep OpensergoClient always be alived.

In current version, when opensergo-control-plane was shutdown , opensergo-java-sdk could not reconnect to opensergo-control-plane. So i improved it.

1、add an enum to defined the status of OpensergoClient.
2、add a keepalive function which was invoked in the start process of OpensergoClient.
3、record and change status for the OpensergoClient instance.

@jnan806 jnan806 force-pushed the opensergoclient_keepalive branch 3 times, most recently from c9d48a2 to 5afed50 Compare October 13, 2022 01:21
@sczyh30 sczyh30 added the kind/enhancement Category issues or PRs related to enhancement label Oct 17, 2022
@jnan806 jnan806 force-pushed the opensergoclient_keepalive branch 4 times, most recently from 5dafb29 to 0d384e5 Compare October 27, 2022 14:23
@sczyh30 sczyh30 linked an issue Nov 4, 2022 that may be closed by this pull request
@jnan806 jnan806 force-pushed the opensergoclient_keepalive branch from 0d384e5 to 38d6971 Compare November 8, 2022 15:35
@jnan806 jnan806 requested a review from sczyh30 November 10, 2022 12:45
Co-authored-by: Eric Zhao <[email protected]>
Co-authored-by: Jiangnan Jia <[email protected]>

Signed-off-by: Jiangnan Jia <[email protected]>
@jnan806 jnan806 force-pushed the opensergoclient_keepalive branch from 38d6971 to b937eb4 Compare November 11, 2022 13:46
@jnan806 jnan806 requested a review from sczyh30 November 11, 2022 13:54
README.md Outdated Show resolved Hide resolved
src/main/java/io/opensergo/OpenSergoClient.java Outdated Show resolved Hide resolved
src/main/java/io/opensergo/OpenSergoClient.java Outdated Show resolved Hide resolved
@jnan806 jnan806 requested a review from sczyh30 November 22, 2022 07:29
@jnan806 jnan806 force-pushed the opensergoclient_keepalive branch from 014f0f6 to a960158 Compare December 3, 2022 14:07
@jnan806 jnan806 force-pushed the opensergoclient_keepalive branch from a960158 to f7ff064 Compare December 3, 2022 14:19
@jnan806 jnan806 marked this pull request as draft February 9, 2023 05:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/enhancement Category issues or PRs related to enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add reconnect mechanism for remote connection shutdown
2 participants