Sailesh Mukil has posted comments on this change. Change subject: IMPALA-3577, IMPALA-3486: Partitions on multiple filesystems breaks with S3_SKIP_INSERT_STAGING ......................................................................
Patch Set 11: (1 comment) http://gerrit.cloudera.org:8080/#/c/3146/11/tests/query_test/test_multiple_filesystems.py File tests/query_test/test_multiple_filesystems.py: Line 63: fs_prefix = "s3a://" + S3_BUCKET_NAME > I was thinking we'd do something even simpler which was to require the Jenk No we don't. We would have had to if we went this way, but I changed it to just use SECONDARY_FILESYSTEM like you mentioned here, so that our builds can choose which filesystem to run as secondary. And we don't need to set S3_BUCKET_NAME or ISILION_NAMENODE. -- To view, visit http://gerrit.cloudera.org:8080/3146 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ib13b610eb9efb68c83894786cea862d7eae43aa7 Gerrit-PatchSet: 11 Gerrit-Project: Impala Gerrit-Branch: cdh5-2.6.0_5.8.0 Gerrit-Owner: Sailesh Mukil <[email protected]> Gerrit-Reviewer: Dan Hecht <[email protected]> Gerrit-Reviewer: Sailesh Mukil <[email protected]> Gerrit-HasComments: Yes
