Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/22833 )
Change subject: IMPALA-12554: (Addendum) Add a flag to not consolidate requests by default ...................................................................... IMPALA-12554: (Addendum) Add a flag to not consolidate requests by default This patch adds a startup flag so that by default the catalog server will not consolidate the grant/revoke requests sent to the Ranger server when there are multiple columns involved in the GRANT/REVOKE statement. Testing: - Added 2 end-to-end tests to make sure the grant/revoke requests sent to the Ranger server would be consolidated only when the flag is explicitly added when we start the catalog server. Change-Id: I4defc59c048be1112380c3a7254ffa8655eee0af Reviewed-on: http://gerrit.cloudera.org:8080/22833 Reviewed-by: Impala Public Jenkins <[email protected]> Tested-by: Impala Public Jenkins <[email protected]> --- M be/src/catalog/catalog.cc M be/src/util/backend-gflag-util.cc M common/thrift/BackendGflags.thrift M fe/src/main/java/org/apache/impala/authorization/ranger/RangerCatalogdAuthorizationManager.java M fe/src/main/java/org/apache/impala/service/BackendConfig.java M tests/authorization/test_ranger.py 6 files changed, 26 insertions(+), 2 deletions(-) Approvals: Impala Public Jenkins: Looks good to me, approved; Verified -- To view, visit http://gerrit.cloudera.org:8080/22833 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I4defc59c048be1112380c3a7254ffa8655eee0af Gerrit-Change-Number: 22833 Gerrit-PatchSet: 4 Gerrit-Owner: Fang-Yu Rao <[email protected]> Gerrit-Reviewer: Aman Sinha <[email protected]> Gerrit-Reviewer: Csaba Ringhofer <[email protected]> Gerrit-Reviewer: Fang-Yu Rao <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Quanlong Huang <[email protected]> Gerrit-Reviewer: Riza Suminto <[email protected]>
