> On July 7, 2017, 7:03 p.m., Sergio Pena wrote: > > sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/ServiceConstants.java > > Lines 128-132 (original), 126-130 (patched) > > <https://reviews.apache.org/r/60502/diff/3/?file=1771260#file1771260line128> > > > > These variables are not used anymore. Do we need them for the new > > SentryHA code?
Those in org.apache.sentry.hdfs.ServiceConstants are not used. Those in org.apache.sentry.service.thrift.ServiceConstants are used. I will remove the unused ones. > On July 7, 2017, 7:03 p.m., Sergio Pena wrote: > > sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/service/thrift/SentryServiceIntegrationBase.java > > Line 84 (original), 83 (patched) > > <https://reviews.apache.org/r/60502/diff/3/?file=1771263#file1771263line84> > > > > This zkServer variable is not used anymore after removing the haEnabled > > code. > > > > Also getZKServer() is not used anymore. Will remove > On July 7, 2017, 7:03 p.m., Sergio Pena wrote: > > sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/minisentry/SentrySrv.java > > Line 66 (original), 66 (patched) > > <https://reviews.apache.org/r/60502/diff/3/?file=1771269#file1771269line66> > > > > This getZKQuorum() method is not used anymore after removing the sentry > > ha. Will we use this in the future for the new sentryha code redesign? will remove. > On July 7, 2017, 7:03 p.m., Sergio Pena wrote: > > sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/minisentry/InternalSentrySrv.java > > Line 222 (original), 211 (patched) > > <https://reviews.apache.org/r/60502/diff/3/?file=1771274#file1771274line222> > > > > This getZKQurom is not used anymore. Should we remove it as well? will remove - Na ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60502/#review179906 ----------------------------------------------------------- On July 7, 2017, 3:11 p.m., Na Li wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/60502/ > ----------------------------------------------------------- > > (Updated July 7, 2017, 3:11 p.m.) > > > Review request for sentry, Alexander Kolbasov, kalyan kumar kalvagadda, > Sergio Pena, and Vamsee Yarlagadda. > > > Repository: sentry > > > Description > ------- > > Remove the definition and usage of SENTRY_HA_ENABLED > > > Diffs > ----- > > > sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/ServiceConstants.java > 83f00ca > > sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/service/thrift/TestSentryServerForPoolWithoutKerberos.java > 62fbb2f > > sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/service/thrift/TestSentryServiceForPoolWithKerberos.java > bd3c1cc > > sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/service/thrift/SentryServiceIntegrationBase.java > dd37e7e > > sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/AbstractTestWithDbProvider.java > d843829 > > sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestPrivilegeWithHAGrantOption.java > 70b48e1 > > sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/ha/TestHaEnd2End.java > 07d74b5 > > sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hive/AbstractTestWithStaticConfiguration.java > 458e91d > > sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/minisentry/InternalSentrySrv.java > 054b193 > > sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/minisentry/SentrySrv.java > dac1151 > > sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/dbprovider/AbstractTestWithDbProvider.java > 17a2d1e > > sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestPrivilegeWithHAGrantOption.java > 53cbd00 > > sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/ha/TestFailover.java > 1c7cd8b > > sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/AbstractTestWithStaticConfiguration.java > fe68f49 > > sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/minisentry/InternalSentrySrv.java > 054b193 > > sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/minisentry/SentrySrv.java > dac1151 > > > Diff: https://reviews.apache.org/r/60502/diff/3/ > > > Testing > ------- > > unit tests > > > Thanks, > > Na Li > >
