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
Improve single protocol handlers #19730
Improve single protocol handlers #19730
Conversation
The job Click to expand the log file-------------------------- -------TEST FAILURE------- -------------------------- [INFO] Results: [INFO] [ERROR] Failures: [ERROR] SingleEncoderDecoderTest.testIncompatibleProtocolSentToClient[supportedProtocol:CLIENT, receivedProtocol:ASD] Expected test to throw an instance of com.hazelcast.internal.server.tcp.ProtocolException [ERROR] SingleEncoderDecoderTest.testIncompatibleProtocolSentToClient[supportedProtocol:CLIENT, receivedProtocol:HZC] Expected test to throw an instance of com.hazelcast.internal.server.tcp.ProtocolException [ERROR] SingleEncoderDecoderTest.testIncompatibleProtocolSentToClient[supportedProtocol:MEMBER, receivedProtocol:ASD] Expected test to throw an instance of com.hazelcast.internal.server.tcp.ProtocolException [ERROR] SingleEncoderDecoderTest.testIncompatibleProtocolSentToClient[supportedProtocol:MEMBER, receivedProtocol:CP2] Expected test to throw an instance of com.hazelcast.internal.server.tcp.ProtocolException [INFO] [ERROR] Tests run: 46505, Failures: 4, Errors: 0, Skipped: 1011 [INFO] |
AdvancedNetworkIntegrationTest#testCompleteMultisocketConfig covers this case because exception thrown in the SingleProtocolDecoder now.
It looks good to me in general. But, I will be waiting for |
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.
Thanks for the fix. LGTM
Thank you for the reviews. |
* Throw exception in the decoder instead of the encoder * Extend missing error handling to TextHandshakeDecoder * Delete SingleEncoderDecoderTest as it's no longer needed. (cherry picked from commit 4224341)
Can one of the admins verify this patch? |
2 similar comments
Can one of the admins verify this patch? |
Can one of the admins verify this patch? |
#18753 failed again because sometimes channel is closed before exception message is sent. I moved exceptions from decoder to encoder so this should be no longer a problem.
I also realized that TextHandshakeDecoder was missing some needed changes after SingleProtocolEncoder introduced. I also included them in this PR.
This PR should be backported to
5.0.z
. I will backport after this PR is merged.Fixes #18753
Fixes https://github.com/hazelcast/hazelcast-mono/issues/1591
EE PR: https://github.com/hazelcast/hazelcast-enterprise/pull/4348
Checklist:
Team:
,Type:
,Source:
,Module:
) and Milestone setAdd to Release Notes
label if changes should be mentioned in release notes orNot Release Notes content
if changes are not relevant for release notes