Internal Jenkins has submitted this change and it was merged. Change subject: IMPALA-3143: Fix permission check when user belongs to supergroup ......................................................................
IMPALA-3143: Fix permission check when user belongs to supergroup This commit fixes an issue where a permission check warning is incorrectly generated when a user that belongs to the supergroup issues a create database statement. The fix is to also check if the user belongs to the supergroup when checking for permissions and ACLs. Testing: This change was tested manually by creating an HDFS directory D that belonged to user/group A and running a CREATE DATABASE foo LOCALTION 'D' statement as a user B that belongs to the 'supergroup' group; the test verified that when the supergroup is taken into account, no permision check warning is issued. Change-Id: I2bb703ac70280c62db404db2c07d87139aa5ae0d Reviewed-on: http://gerrit.cloudera.org:8080/3164 Reviewed-by: Alex Behm <[email protected]> Tested-by: Internal Jenkins --- M fe/src/main/java/com/cloudera/impala/util/FsPermissionChecker.java 1 file changed, 10 insertions(+), 3 deletions(-) Approvals: Internal Jenkins: Verified Alex Behm: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/3164 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2bb703ac70280c62db404db2c07d87139aa5ae0d Gerrit-PatchSet: 3 Gerrit-Project: Impala Gerrit-Branch: cdh5-2.6.0_5.8.0 Gerrit-Owner: Dimitris Tsirogiannis <[email protected]> Gerrit-Reviewer: Alex Behm <[email protected]> Gerrit-Reviewer: Dimitris Tsirogiannis <[email protected]> Gerrit-Reviewer: Internal Jenkins
