Zoltan Chovan has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/23099 )

Change subject: [tests] fix flakiness in exactly_once_writes-itest
......................................................................


Patch Set 2:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/23099/2/src/kudu/integration-tests/exactly_once_writes-itest.cc
File src/kudu/integration-tests/exactly_once_writes-itest.cc:

http://gerrit.cloudera.org:8080/#/c/23099/2/src/kudu/integration-tests/exactly_once_writes-itest.cc@104
PS2, Line 104: const vector<string>& master_flags,
             :       const vector<string>& ts_flags,
Just out of curiosity, what was the reason for flipping the parameter order? 
Only readability?



--
To view, visit http://gerrit.cloudera.org:8080/23099
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ia2d9da084220fa48749fa67cfb4dca422a9601f2
Gerrit-Change-Number: 23099
Gerrit-PatchSet: 2
Gerrit-Owner: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Abhishek Chennaka <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: KeDeng <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Marton Greber <[email protected]>
Gerrit-Reviewer: Zoltan Chovan <[email protected]>
Gerrit-Comment-Date: Fri, 27 Jun 2025 14:03:15 +0000
Gerrit-HasComments: Yes

Reply via email to