----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51773/#review148375 -----------------------------------------------------------
Fix it, then Ship it! sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/TestUriPermissions.java (line 162) <https://reviews.apache.org/r/51773/#comment215834> Same here sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/TestUriPermissions.java (line 201) <https://reviews.apache.org/r/51773/#comment215833> remove space - Hao Hao On Sept. 9, 2016, 9:26 p.m., Anne Yu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/51773/ > ----------------------------------------------------------- > > (Updated Sept. 9, 2016, 9:26 p.m.) > > > Review request for sentry and Hao Hao. > > > Bugs: SENTRY-1464 > https://issues.apache.org/jira/browse/SENTRY-1464 > > > Repository: sentry > > > Description > ------- > > Fixed UnmanagedHiveServer2 parse property API so it doesn't always add prefix > "hdfs" in Hivewarehouse Uri; > Print out queries so that can debug straightforward what's going wrong with > most recent test failures on s3 cluster. > > > Diffs > ----- > > > sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/TestMetadataPermissions.java > 524fa1c9db4c466e7aa2a8b16b426a61abad31bf > > sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbCrossOperations.java > 62c0f58dc093df09404c9e12a4203120d420fc6c > > sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/AbstractTestWithStaticConfiguration.java > ba29ca39a4e457f628a12e7f49b8d8dbe04341fc > > sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/Context.java > a92d8c9c75110c54963e0b2ffca3b7e44548e9e4 > > sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/TestExportImportPrivileges.java > a02a156d6fbfc49df7d248bc4258510c48395ee2 > > sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/TestOperationsPart2.java > e11a817f2a41ceb67e8b3d8fe7ea231dbba7f968 > > sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/TestSandboxOps.java > 202173c14366e57c7f9109b9e0c75a2213cfbc63 > > sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/TestUriPermissions.java > a2850b987d8734a69291a7fb90b0e95ce7143618 > > sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/fs/TestHiveWarehouseOnExtFs.java > ed965c1ec4639ba42175a61cd1087b928d7bb922 > > sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/hiveserver/UnmanagedHiveServer.java > d5c5db40a9d42140c1f55d3665fd152151d7285e > > Diff: https://reviews.apache.org/r/51773/diff/ > > > Testing > ------- > > On a real cluster. > > > Thanks, > > Anne Yu > >
