Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/17514 )
Change subject: [test] minor clean up on TestUnsafeChangeConfigLeaderWithPendingConfig ...................................................................... Patch Set 1: Code-Review+2 This change looks good to me. I guess Bankim's +2 is needed to merge into the 1.15.x branch. If Bankim doesn't like removing those INFO logs, I guess we can add his -2 on this patch instead :) I'm totally fine with that -- this patch is just a clean-up and an attempt to have more information to troubleshoot a test if it fails again; there isn't any user-facing functionality. -- To view, visit http://gerrit.cloudera.org:8080/17514 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: branch-1.15.x Gerrit-MessageType: comment Gerrit-Change-Id: Id2bf354764f3ddb371de19910c5f879c1a06c78b Gerrit-Change-Number: 17514 Gerrit-PatchSet: 1 Gerrit-Owner: Grant Henke <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Comment-Date: Wed, 26 May 2021 03:11:58 +0000 Gerrit-HasComments: No
