Fredy Wijaya has uploaded a new patch set (#5). ( http://gerrit.cloudera.org:8080/11721 )
Change subject: IMPALA-7721: Fix broken /catalog_object web API when getting a privilege ...................................................................... IMPALA-7721: Fix broken /catalog_object web API when getting a privilege Before this patch, /catalog_object web API was broken when getting a privilege due to an incorrect way of getting a role ID. IMPALA-7616 broke this even more due to a lack of test coverage in /catalog_object when authorization is enabled. This patch fixes the issue and makes the /catalog_object web API usable again for getting a privilege. Testing: - Added a new BE test - Added a new E2E test - Ran all E2E authorization tests Change-Id: I525149d113a1437c1e1493ad3c25a755e370b0c7 --- M be/src/catalog/catalog-util-test.cc M be/src/catalog/catalog-util.cc M be/src/catalog/catalog-util.h M fe/src/main/java/org/apache/impala/catalog/Catalog.java M fe/src/main/java/org/apache/impala/catalog/PrincipalPrivilege.java M tests/authorization/test_authorization.py 6 files changed, 232 insertions(+), 13 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/21/11721/5 -- To view, visit http://gerrit.cloudera.org:8080/11721 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newpatchset Gerrit-Change-Id: I525149d113a1437c1e1493ad3c25a755e370b0c7 Gerrit-Change-Number: 11721 Gerrit-PatchSet: 5 Gerrit-Owner: Fredy Wijaya <fwij...@cloudera.com> Gerrit-Reviewer: Csaba Ringhofer <csringho...@cloudera.com> Gerrit-Reviewer: Fredy Wijaya <fwij...@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Gerrit-Reviewer: Vuk Ercegovac <vercego...@cloudera.com>