>From Janhavi Tripurwar <[email protected]>: Attention is currently required from: Michael Blow.
Janhavi Tripurwar has posted comments on this change by Janhavi Tripurwar. ( https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/21168?usp=email ) Change subject: [ASTERIXDB-3764][NET]: Avoid reusing failed connections after NC restart ...................................................................... Patch Set 3: Code-Review+1 Verified+1 (4 comments) Patchset: PS2: > It seems reasonable- do you believe this is what caused the behavior observed > in the tests? If so, w […] This is a race condition and could explain the test failure. We should take this into Trinity for 7.6.12. Commit Message: https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/21168/comment/defe9186_7101b9f9?usp=email : PS2, Line 10: - Remove failed connections from outgoingConnectionMap before creating new ones. : - Expose hasConnectionFailure() in MultiplexedConnection. : - Ensure only the current connection is removed from outgoingConnectionMap on close. > please add newlines to stay within 72 chars Done File hyracks-fullstack/hyracks/hyracks-net/src/main/java/org/apache/hyracks/net/protocols/muxdemux/MultiplexedConnection.java: https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/21168/comment/d83c2814_0b6ff433?usp=email : PS2, Line 100: public > can we move the test to the same package as this to avoid having to make this > `public`? Done File hyracks-fullstack/hyracks/hyracks-net/src/test/java/org/apache/hyracks/net/tests/MuxDemuxReconnectTest.java: https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/21168/comment/e5187631_9b53ce11?usp=email : PS2, Line 19: org.apache.hyracks.net.tests > ```suggestion […] Done -- To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/21168?usp=email To unsubscribe, or for help writing mail filters, visit https://asterix-gerrit.ics.uci.edu/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: asterixdb Gerrit-Branch: lumina Gerrit-Change-Id: If76cee14335deffa14c67982a02fe9f0f8aad944 Gerrit-Change-Number: 21168 Gerrit-PatchSet: 3 Gerrit-Owner: Janhavi Tripurwar <[email protected]> Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Janhavi Tripurwar <[email protected]> Gerrit-Reviewer: Jenkins <[email protected]> Gerrit-CC: Michael Blow <[email protected]> Gerrit-Attention: Michael Blow <[email protected]> Gerrit-Comment-Date: Thu, 30 Apr 2026 10:22:20 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: Yes Comment-In-Reply-To: Michael Blow <[email protected]>
