Dimitris Tsirogiannis has posted comments on this change.

Change subject: IMPALA-3143: Fix permission check when user belongs to 
supergroup
......................................................................


Patch Set 1:

Unfortunately, it's not that simple as changing a test in 
TestPermissionValidation(). We need to: a) create a path the owner of which is 
not the current user (that runs the tests) and b) make sure the current user 
belongs to the "supergroup" group, which based on what I've seen, is not the 
default setup.

-- 
To view, visit http://gerrit.cloudera.org:8080/3164
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I2bb703ac70280c62db404db2c07d87139aa5ae0d
Gerrit-PatchSet: 1
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-HasComments: No

Reply via email to