Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/21940 )
Change subject: IMPALA-12554: Create one Ranger policy for multi-column GRANT ...................................................................... IMPALA-12554: Create one Ranger policy for multi-column GRANT This patch makes Impala create only one Ranger policy for the GRANT statement when there are multiple columns specified to reduce the number of policies created on the Ranger server. Note that this patch relies on RANGER-4585 and RANGER-4638. Testing: - Manually verified that Impala's catalog daemon only sends one GrantRevokeRequest to the Ranger plug-in and that the value of the key 'column' is a comma-separated list of column names involved in the GRANT statement. - Added an end-to-end test to verify only one Ranger policy will be created in a multi-column GRANT statement. Change-Id: I2b0ebba256c7135b4b0d2160856202292d720c6d Reviewed-on: http://gerrit.cloudera.org:8080/21940 Reviewed-by: Impala Public Jenkins <[email protected]> Tested-by: Impala Public Jenkins <[email protected]> --- M fe/src/main/java/org/apache/impala/authorization/ranger/RangerCatalogdAuthorizationManager.java M fe/src/main/java/org/apache/impala/authorization/ranger/RangerImpalaResourceBuilder.java M tests/authorization/test_ranger.py 3 files changed, 128 insertions(+), 3 deletions(-) Approvals: Impala Public Jenkins: Looks good to me, approved; Verified -- To view, visit http://gerrit.cloudera.org:8080/21940 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I2b0ebba256c7135b4b0d2160856202292d720c6d Gerrit-Change-Number: 21940 Gerrit-PatchSet: 4 Gerrit-Owner: Fang-Yu Rao <[email protected]> Gerrit-Reviewer: Aman Sinha <[email protected]> Gerrit-Reviewer: Csaba Ringhofer <[email protected]> Gerrit-Reviewer: Daniel Becker <[email protected]> Gerrit-Reviewer: Fang-Yu Rao <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Quanlong Huang <[email protected]>
