----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15038/#review28217 -----------------------------------------------------------
Ship it! Ship It! - Shreepadma Venugopalan On Oct. 29, 2013, 6:22 p.m., Sravya Tirukkovalur wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/15038/ > ----------------------------------------------------------- > > (Updated Oct. 29, 2013, 6:22 p.m.) > > > Review request for sentry and Shreepadma Venugopalan. > > > Bugs: SENTRY-52 > https://issues.apache.org/jira/browse/SENTRY-52 > > > Repository: sentry > > > Description > ------- > > This was causing a couple of failures in the e2e tests. > > //This prints hdfs://host:port/path > path = new Path(fileSystem.getUri() + policyDir); > fileSystem.mkdirs(path); > System.out.println(path); > > //Whereas this prints just /path > path = new Path(policyDir); > fileSystem.mkdirs(path); > System.out.println(path); > > > Diffs > ----- > > > sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/fs/ClusterDFS.java > 3641d04 > > Diff: https://reviews.apache.org/r/15038/diff/ > > > Testing > ------- > > Tests which use this path in URIs now pass. > > > Thanks, > > Sravya Tirukkovalur > >
