Skip to content

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
@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