Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/23045 )
Change subject: Add REST API integration tests ...................................................................... Patch Set 11: > The assumption was that calls from REST API are transparently > forwarded to the authz logic that already exists. Aside from > analysing just the code, we've figured that we should probably do > at least "smoke" type of tests in terms of the authz integration > tests for REST API. The only thing that came up is the principal > name mapping issue, but that has been resolved by Gabi. The thought > behind the creation of this patch was along these lines. > > LGTM, but let's see if Alexey has any more observations. Thanks > Alexey for taking a closer look! That makes sense to me -- thank you for the extra details, Marton. As of PS11, this patch looks good to me. -- To view, visit http://gerrit.cloudera.org:8080/23045 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ifd3ff0dfd67cbc2b5ed0454372dd2bcea71e2ba3 Gerrit-Change-Number: 23045 Gerrit-PatchSet: 11 Gerrit-Owner: Gabriella Lotz <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Gabriella Lotz <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Marton Greber <[email protected]> Gerrit-Reviewer: Zoltan Chovan <[email protected]> Gerrit-Comment-Date: Mon, 25 Aug 2025 15:02:23 +0000 Gerrit-HasComments: No
