Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/14990 )
Change subject: KUDU-3011 p4: follow-up to 4619473 ...................................................................... Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/14990/1//COMMIT_MSG Commit Message: http://gerrit.cloudera.org:8080/#/c/14990/1//COMMIT_MSG@9 PS1, Line 9: This addresses feedback Thank you for addressing the feedback! 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? -- 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: 1 Gerrit-Owner: Andrew Wong <aw...@cloudera.com> Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Comment-Date: Wed, 08 Jan 2020 03:18:10 +0000 Gerrit-HasComments: Yes