Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/14990 )
Change subject: KUDU-3011 p4: follow-up to 4619473 ...................................................................... Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/14990/1/src/kudu/integration-tests/tablet_server_quiescing-itest.cc File src/kudu/integration-tests/tablet_server_quiescing-itest.cc: http://gerrit.cloudera.org:8080/#/c/14990/1/src/kudu/integration-tests/tablet_server_quiescing-itest.cc@390 PS1, Line 390: MonoTime::Now() < past_original_expiration > With this condition, is this cycle going to be run at least once? Oops :) -- To view, visit http://gerrit.cloudera.org:8080/14990 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I6d30b28fec9e3c34060f3ad45f6e34a33d4e9a23 Gerrit-Change-Number: 14990 Gerrit-PatchSet: 2 Gerrit-Owner: Andrew Wong <aw...@cloudera.com> Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com> Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Comment-Date: Wed, 08 Jan 2020 03:21:02 +0000 Gerrit-HasComments: Yes