Attila Bukor has posted comments on this change. ( http://gerrit.cloudera.org:8080/21399 )
Change subject: [security-flags-itest] Fix missing command line flags ...................................................................... Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/21399/1/src/kudu/integration-tests/security-flags-itest.cc File src/kudu/integration-tests/security-flags-itest.cc: http://gerrit.cloudera.org:8080/#/c/21399/1/src/kudu/integration-tests/security-flags-itest.cc@54 PS1, Line 54: ASSERT_EQ > The whole point of this test is that we can find the flag with its string n Good catch, thanks for verifying this. -- To view, visit http://gerrit.cloudera.org:8080/21399 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Iec751e8761562612d97b886740c9b20cd134a0bc Gerrit-Change-Number: 21399 Gerrit-PatchSet: 2 Gerrit-Owner: Ádám Bakai <aba...@cloudera.com> Gerrit-Reviewer: Alexey Serbin <ale...@apache.org> Gerrit-Reviewer: Anonymous Coward (763) Gerrit-Reviewer: Attila Bukor <abu...@apache.org> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Mahesh Reddy <mre...@cloudera.com> Gerrit-Reviewer: Wang Xixu <1450306...@qq.com> Gerrit-Reviewer: Ádám Bakai <aba...@cloudera.com> Gerrit-Comment-Date: Wed, 08 May 2024 10:11:38 +0000 Gerrit-HasComments: Yes
