> On Sept. 10, 2013, 9:51 p.m., Shreepadma Venugopalan wrote: > > sentry-tests/src/test/java/org/apache/sentry/tests/e2e/hiveserver/UnmanagedHiveServer.java, > > line 47 > > <https://reviews.apache.org/r/14070/diff/1/?file=350373#file350373line47> > > > > public static final driverName?
Looks like the HiveDriver test is being made in HiveServerFactory, so removing here. - Sravya ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14070/#review26021 ----------------------------------------------------------- On Sept. 10, 2013, 9:36 p.m., Sravya Tirukkovalur wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/14070/ > ----------------------------------------------------------- > > (Updated Sept. 10, 2013, 9:36 p.m.) > > > Review request for sentry, Prasad Mujumdar and Shreepadma Venugopalan. > > > Repository: sentry > > > Description > ------- > > This is required as we need to authenticate before a connection is created > when using UnmanagedHiveServer > > > Diffs > ----- > > sentry-tests/src/test/java/org/apache/sentry/tests/e2e/Context.java 30d1aaa > > sentry-tests/src/test/java/org/apache/sentry/tests/e2e/hiveserver/AbstractHiveServer.java > 0c2fbc3 > > sentry-tests/src/test/java/org/apache/sentry/tests/e2e/hiveserver/EmbeddedHiveServer.java > 48c112c > > sentry-tests/src/test/java/org/apache/sentry/tests/e2e/hiveserver/HiveServer.java > ce4b3e8 > > sentry-tests/src/test/java/org/apache/sentry/tests/e2e/hiveserver/HiveServerFactory.java > af3344e > > sentry-tests/src/test/java/org/apache/sentry/tests/e2e/hiveserver/UnmanagedHiveServer.java > 288d2f7 > > Diff: https://reviews.apache.org/r/14070/diff/ > > > Testing > ------- > > All tests pass > > > Thanks, > > Sravya Tirukkovalur > >
