Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/15174 )
Change subject: [rpc-test] fix flaky TestCancellation ...................................................................... Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/15174/1/src/kudu/rpc/rpc-test-base.h File src/kudu/rpc/rpc-test-base.h: http://gerrit.cloudera.org:8080/#/c/15174/1/src/kudu/rpc/rpc-test-base.h@570 PS1, Line 570: EXPECT_TRUE(s.IsAborted() || s.IsTimedOut()) nit: could you add ' << s.ToString()' ? That can help in troubleshooting in case if this assertion ever fails. -- To view, visit http://gerrit.cloudera.org:8080/15174 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: If140a9215c94d781c2c61ec30b18d2d67d80310a Gerrit-Change-Number: 15174 Gerrit-PatchSet: 1 Gerrit-Owner: Thomas Tauber-Marshall <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Comment-Date: Thu, 06 Feb 2020 21:29:13 +0000 Gerrit-HasComments: Yes
