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
ISPN-15239 HotRod client repeatedly retries iteration operation again… #11422
Conversation
@@ -112,7 +113,7 @@ protected long handleNextResponse(IterationNextResponse<K, E> nextResponse, Map. | |||
|
|||
@Override | |||
protected void handleThrowableInResponse(Throwable t, Map.Entry<SocketAddress, IntSet> target) { | |||
if (t instanceof TransportException || t instanceof RemoteIllegalLifecycleStateException) { | |||
if (t instanceof TransportException || t instanceof RemoteIllegalLifecycleStateException || t instanceof ConnectException) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wondering if it needs all of:
Lines 160 to 163 in 7534ad3
if (cause instanceof RemoteIllegalLifecycleStateException | |
|| cause instanceof IOException | |
|| cause instanceof TransportException | |
|| cause instanceof RejectedExecutionException) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hrmm, only ConnectException would be better if it is suitable.
@@ -112,7 +113,7 @@ protected long handleNextResponse(IterationNextResponse<K, E> nextResponse, Map. | |||
|
|||
@Override | |||
protected void handleThrowableInResponse(Throwable t, Map.Entry<SocketAddress, IntSet> target) { | |||
if (t instanceof TransportException || t instanceof RemoteIllegalLifecycleStateException) { | |||
if (t instanceof TransportException || t instanceof RemoteIllegalLifecycleStateException || t instanceof ConnectException) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hrmm, only ConnectException would be better if it is suitable.
@@ -112,7 +113,7 @@ protected long handleNextResponse(IterationNextResponse<K, E> nextResponse, Map. | |||
|
|||
@Override | |||
protected void handleThrowableInResponse(Throwable t, Map.Entry<SocketAddress, IntSet> target) { | |||
if (t instanceof TransportException || t instanceof RemoteIllegalLifecycleStateException) { | |||
if (t instanceof TransportException || t instanceof RemoteIllegalLifecycleStateException || t instanceof ConnectException) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We also need to add this to the hotrod module as well.
…st failed server after removal from topology
Updated the hotrod module. |
Integrated into main, thanks @jabolina ! |
…st failed server after removal from topology
https://issues.redhat.com/browse/ISPN-15239
Looking at the logs, the iteration operation starts and calculates the segment owners, and then the topology updates. So, I updated the exception handling part.