Harrison Sheinblatt has posted comments on this change. Change subject: Disable HDFS ACLs on the minicluster ......................................................................
Patch Set 1: I think we can't make this change globally as we'd drop test coverage. I think we need to put this in as a workaround temporarily only for data load, but for testing leave the acls on. Not sure this is complete or referring to the right setting though, so still need to double check: $ find . -type f | xargs grep -i "Skip.*hdfs_acls" ./metadata/test_hdfs_permissions.py:@SkipIfS3.hdfs_acls ./query_test/test_insert_behaviour.py: @SkipIfS3.hdfs_acls ./query_test/test_insert_behaviour.py: @SkipIfIsilon.hdfs_acls ./query_test/test_insert_behaviour.py: @SkipIfS3.hdfs_acls ./query_test/test_insert_behaviour.py: @SkipIfIsilon.hdfs_acls ./query_test/test_insert_behaviour.py: @SkipIfS3.hdfs_acls ./query_test/test_insert_behaviour.py: @SkipIfIsilon.hdfs_acls ./query_test/test_insert_behaviour.py: @SkipIfS3.hdfs_acls ./query_test/test_insert_behaviour.py: @SkipIfIsilon.hdfs_acls ./query_test/test_insert_behaviour.py: @SkipIfS3.hdfs_acls ./query_test/test_insert_behaviour.py: @SkipIfIsilon.hdfs_acls ./custom_cluster/test_insert_behaviour.py:@SkipIfS3.hdfs_acls -- To view, visit http://gerrit.cloudera.org:8080/5350 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I1799ef34dca3b94acdfcde1fe7e5d143bdfd24c7 Gerrit-PatchSet: 1 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: Taras Bobrovytsky <tbobrovyt...@cloudera.com> Gerrit-Reviewer: Harrison Sheinblatt <h...@hotmail.com> Gerrit-HasComments: No