Joe McDonnell has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/14334 )
Change subject: IMPALA-8960: Fix test_owner_privileges::test_drop_if_exists on S3 ...................................................................... IMPALA-8960: Fix test_owner_privileges::test_drop_if_exists on S3 hdfs:// URIs do not work on Impala's S3 tests. The fix is to drop the hdfs:// part of the URI. Our tests run against the default filesystem, so dropping the hdfs:// works on both HDFS test setups and S3 test setups. Testing: - Ran tests on S3 - Ran test_owner_privileges.py on HDFS locally Change-Id: Ibfe15ac2d5ba0d8a6d4383be8d01395c74d67332 Reviewed-on: http://gerrit.cloudera.org:8080/14334 Reviewed-by: Michael Ho <k...@cloudera.com> Tested-by: Impala Public Jenkins <impala-public-jenk...@cloudera.com> --- M tests/authorization/test_owner_privileges.py M tests/common/sentry_cache_test_suite.py 2 files changed, 6 insertions(+), 5 deletions(-) Approvals: Michael Ho: Looks good to me, approved Impala Public Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/14334 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Ibfe15ac2d5ba0d8a6d4383be8d01395c74d67332 Gerrit-Change-Number: 14334 Gerrit-PatchSet: 4 Gerrit-Owner: Joe McDonnell <joemcdonn...@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Gerrit-Reviewer: Joe McDonnell <joemcdonn...@cloudera.com> Gerrit-Reviewer: Michael Ho <k...@cloudera.com>
