> On July 25, 2014, 9:38 p.m., Sravya Tirukkovalur wrote: > > sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/persistent/SentryStore.java, > > line 1011 > > <https://reviews.apache.org/r/23600/diff/3-4/?file=641483#file641483line1011> > > > > Servername should never be null? > > Arun Suresh wrote: > Hmmm... it doesn't imply that serverName can be null.. Im just doing a > safety check (I do a fromNULLCol for all columns in the aggregated unique > key.. so thought i might as well put it for serverName as well)
ok, sounds fine. > On July 25, 2014, 9:38 p.m., Sravya Tirukkovalur wrote: > > sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/persistent/SentryStore.java, > > line 1030 > > <https://reviews.apache.org/r/23600/diff/3-4/?file=641483#file641483line1030> > > > > Why is this changed back? > > Arun Suresh wrote: > So.. this is diff between 2 versions of my diff... if was always > "safeTrim".. and doing a "safeTrimLower" had caused 1 test case to fail. so > removing it since i didn't want to touch the testcase.. Makes sense, will it be possible for you to create a follow on bug just to track this please? Btw, thanks for catching this. - Sravya ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23600/#review48777 ----------------------------------------------------------- On July 25, 2014, 9:34 p.m., Arun Suresh wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/23600/ > ----------------------------------------------------------- > > (Updated July 25, 2014, 9:34 p.m.) > > > Review request for sentry, Jarek Cecho, Prasad Mujumdar, and Sravya > Tirukkovalur. > > > Repository: sentry > > > Description > ------- > > SENTRY-339 : > > Remove the need to construct a privilegeName to check for uniqueness. > - Remove the PrivilegeName column > - Remove the constructPrivilegeName() method > - Updated the sql scripts > > > Diffs > ----- > > > sentry-provider/sentry-provider-db/src/gen/thrift/gen-javabean/org/apache/sentry/provider/db/service/thrift/TSentryPrivilege.java > 9e8ac4c > > sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/model/MSentryPrivilege.java > f8491db > > sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/model/package.jdo > 945227e > > sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/persistent/SentryStore.java > ff8acdc > > sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/thrift/SentryPolicyStoreProcessor.java > 3bb7285 > sentry-provider/sentry-provider-db/src/main/resources/sentry-db2-1.4.0.sql > f2a62d2 > > sentry-provider/sentry-provider-db/src/main/resources/sentry-derby-1.4.0.sql > f2a62d2 > > sentry-provider/sentry-provider-db/src/main/resources/sentry-mysql-1.4.0.sql > 70f4dbb > > sentry-provider/sentry-provider-db/src/main/resources/sentry-oracle-1.4.0.sql > 363590e > > sentry-provider/sentry-provider-db/src/main/resources/sentry-postgres-1.4.0.sql > 5dfae03 > > sentry-provider/sentry-provider-db/src/main/resources/sentry_policy_service.thrift > fdc7b9c > > sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/service/persistent/TestSentryStore.java > 7637376 > > sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/service/thrift/TestSentryServerWithoutKerberos.java > 79579c6 > > sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/service/thrift/TestSentryServiceIntegration.java > a4ae291 > > sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/TestPolicyImport.java > 948b0c4 > > Diff: https://reviews.apache.org/r/23600/diff/ > > > Testing > ------- > > Ran all e2e tests and unit tests. > > > Thanks, > > Arun Suresh > >
