Quanlong Huang has uploaded this change for review. ( http://gerrit.cloudera.org:8080/24510
Change subject: IMPALA-13946: Turn on allow_catalog_cache_op_from_masked_users by default ...................................................................... IMPALA-13946: Turn on allow_catalog_cache_op_from_masked_users by default Most of the users turn this on to resolve the performance regression on INVALIDATE and REFRESH commands. The regression is due to additional metadata loading if the table is unloaded in catalogd. Though this is resolved by IMPALA-14703, i.e. don't need metadata loading anymore, checking table masking policies is still an additional work. Setting allow_catalog_cache_op_from_masked_users to true skips this check. Test Changes: - Moved the negative test (test_block_metadata_update) to TestRangerIndependent since it now requires restarting the cluster with the flag off. Also mirrored it to test on both catalog modes. - Moved the positive tests, test_allow_metadata_update_legacy_catalog and test_allow_metadata_update_local_catalog to TestRangerLocalCatalog and TestRangerLegacyCatalog respectively since they don't need cluster restart now. - For FE tests that depend on this flag to be false, explicitly set it before the tests. Tests - Ran CORE tests Change-Id: I2d3be4467c6dddf5d9cbe1f68e1a19555b3f9359 --- M be/src/service/frontend.cc M fe/src/main/java/org/apache/impala/service/BackendConfig.java M fe/src/test/java/org/apache/impala/authorization/AuthorizationStmtTest.java M fe/src/test/java/org/apache/impala/authorization/ranger/RangerAuditLogTest.java M tests/authorization/test_ranger.py 5 files changed, 62 insertions(+), 40 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/10/24510/1 -- To view, visit http://gerrit.cloudera.org:8080/24510 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: I2d3be4467c6dddf5d9cbe1f68e1a19555b3f9359 Gerrit-Change-Number: 24510 Gerrit-PatchSet: 1 Gerrit-Owner: Quanlong Huang <[email protected]>
