Zoltan Chovan has posted comments on this change. ( http://gerrit.cloudera.org:8080/19758 )
Change subject: Update security-itest jwt tests ...................................................................... Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/19758/1/src/kudu/integration-tests/security-itest.cc File src/kudu/integration-tests/security-itest.cc: http://gerrit.cloudera.org:8080/#/c/19758/1/src/kudu/integration-tests/security-itest.cc@657 PS1, Line 657: ASSERT_FALSE(s.ok()); > Here and below: while you are it, consider adding a check for specific type Done -- To view, visit http://gerrit.cloudera.org:8080/19758 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ie6727ec5c4fee6ad791b9ab5446f31ee2bf97c22 Gerrit-Change-Number: 19758 Gerrit-PatchSet: 2 Gerrit-Owner: Zoltan Chovan <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Attila Bukor <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Zoltan Chovan <[email protected]> Gerrit-Comment-Date: Mon, 17 Apr 2023 19:34:28 +0000 Gerrit-HasComments: Yes
