> On Aug. 24, 2016, 1:54 a.m., Hao Hao wrote: > > sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/TestExportImportPrivileges.java, > > line 134 > > <https://reviews.apache.org/r/51353/diff/1/?file=1482631#file1482631line134> > > > > Could you please comment on why we need to set this property?
Import/Export works with s3 storage system only when this is turned on. CDH-42473. I will also comment in the code. - Anne ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51353/#review146605 ----------------------------------------------------------- On Aug. 23, 2016, 11:42 p.m., Anne Yu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/51353/ > ----------------------------------------------------------- > > (Updated Aug. 23, 2016, 11:42 p.m.) > > > Review request for sentry and Hao Hao. > > > Bugs: SENTRY-1447 > https://issues.apache.org/jira/browse/SENTRY-1447 > > > Repository: sentry > > > Description > ------- > > We start to see more e2e test failures on s3 clusters. For example, > org.apache.sentry.tests.e2e.hdfs.TestDbHdfsExtMaxGroups need to be ignored > for Sentry on S3; > If run > org.apache.sentry.tests.e2e.hive.fs.TestHiveWarehouseOnExtFs.TestCreateTable, > have to correctly configure "-Dsentry.e2etest.storage.uri=s3a://sentry-s3"; > Add exec(statement, "set hive.exim.uri.scheme.whitelist=hdfs,pfile,s3a;") for > import/export tests. > Also fixed on long test class causes time out failure. > > > Diffs > ----- > > sentry-tests/sentry-tests-hive/pom.xml > 2979b45810b04665c2c66275a0b8544b5f2065d8 > > sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestColumnEndToEnd.java > 8c152576d452b3069258e7f95c93cbf42e34e325 > > sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbCrossOperations.java > PRE-CREATION > > sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hdfs/TestDbHdfsBase.java > e545c37c182f3d77995323659ae539a8ad1de0cb > > sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/TestExportImportPrivileges.java > b38eaaf35ec97978fef06fe91e3e5f5e391d4cc3 > > sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/TestOperationsPart2.java > 7186f1af8159c30064e5ae3b5b1900d4556e1e4a > > Diff: https://reviews.apache.org/r/51353/diff/ > > > Testing > ------- > > On a test cluster. > > > Thanks, > > Anne Yu > >