Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/17504 )
Change subject: [test] minor clean up on TestUnsafeChangeConfigLeaderWithPendingConfig ...................................................................... Patch Set 1: > > Patch Set 1: > > > > > > Patch Set 1: > > > > > > > > > Why remove those info log lines? > > > > > > > > Because they are useless in an automated tests. They might > make > > > sense during developing of the test, but there isn't much > sense > > > keeping them since nobody looks at them during regular runs. > If a > > > test scenario fails, the information in the assert messages > should > > > be enough to start troubleshooting. > > > > > > They could help debug in case of a test failure. I'm not > convinced > > > removing log lines is the right thing to do unless it's > causing log > > > spew and making debugging even more difficult. > > > > Nope, they could not. As you can see, many of those duplicates > what's output with assertions, and for the case I was looking at > they are completely useless. > > It may not help in this case but consider an example of someone new > trying to debug a test issue, instead of trying to co-relate with > the code such log messages can serve as marker points. > > I won't try to argue further. One of those cases where we can agree > to disagree and move on :). OK, this makes sense to me. -- To view, visit http://gerrit.cloudera.org:8080/17504 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Id2bf354764f3ddb371de19910c5f879c1a06c78b Gerrit-Change-Number: 17504 Gerrit-PatchSet: 1 Gerrit-Owner: Alexey Serbin <aser...@cloudera.com> Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com> Gerrit-Reviewer: Bankim Bhavsar <ban...@cloudera.com> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Mahesh Reddy <mre...@cloudera.com> Gerrit-Comment-Date: Wed, 26 May 2021 02:12:43 +0000 Gerrit-HasComments: No