Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/12292 )
Change subject: IMPALA-6479: Update DESCRIBE to respect column privileges ...................................................................... IMPALA-6479: Update DESCRIBE to respect column privileges Modified the Frontend to filter columns from the DESCRIBE statement. Additionally, if a user has select on at least one column, they can run DESCRIBE and see most metadata. If they do not have full table access, they will not see location or view query metadata. Testing: Added tests to validate users that have one or more column access can run describe and that the output is filtered accordingly. Change-Id: Ic96ae184fccdc88ba970b5adcd501da1966accb9 Reviewed-on: http://gerrit.cloudera.org:8080/9276 Reviewed-by: Alex Behm <[email protected]> Tested-by: Impala Public Jenkins Reviewed-on: http://gerrit.cloudera.org:8080/12292 Reviewed-by: Fredy Wijaya <[email protected]> Tested-by: Impala Public Jenkins <[email protected]> --- M be/src/service/client-request-state.cc M be/src/service/frontend.cc M be/src/service/frontend.h M common/thrift/Frontend.thrift M fe/src/main/java/org/apache/impala/analysis/DescribeTableStmt.java M fe/src/main/java/org/apache/impala/analysis/InsertStmt.java M fe/src/main/java/org/apache/impala/catalog/Column.java M fe/src/main/java/org/apache/impala/catalog/Table.java M fe/src/main/java/org/apache/impala/service/DescribeResultFactory.java M fe/src/main/java/org/apache/impala/service/Frontend.java M fe/src/main/java/org/apache/impala/service/JniFrontend.java M fe/src/test/java/org/apache/impala/analysis/AuditingTest.java M fe/src/test/java/org/apache/impala/analysis/AuthorizationTest.java 13 files changed, 331 insertions(+), 69 deletions(-) Approvals: Fredy Wijaya: Looks good to me, approved Impala Public Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/12292 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: 2.x Gerrit-MessageType: merged Gerrit-Change-Id: Ic96ae184fccdc88ba970b5adcd501da1966accb9 Gerrit-Change-Number: 12292 Gerrit-PatchSet: 3 Gerrit-Owner: Quanlong Huang <[email protected]> Gerrit-Reviewer: Adam Holley <[email protected]> Gerrit-Reviewer: Alex Behm <[email protected]> Gerrit-Reviewer: Bharath Vissapragada <[email protected]> Gerrit-Reviewer: Fredy Wijaya <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Quanlong Huang <[email protected]>
