> On Feb. 8, 2017, 2:19 p.m., kalyan kumar kalvagadda wrote: > > sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/authorizer/DefaultSentryValidator.java, > > line 213 > > <https://reviews.apache.org/r/56411/diff/2/?file=1627253#file1627253line213> > > > > Just curious to know. Why didn;t the build fail when changes for > > SENTRY-1577 comitted?
I think it is because we do not build with hive v2 by default. - Nachiket ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56411/#review164684 ----------------------------------------------------------- On Feb. 8, 2017, 5:15 a.m., Alexander Kolbasov wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/56411/ > ----------------------------------------------------------- > > (Updated Feb. 8, 2017, 5:15 a.m.) > > > Review request for sentry, Aihua Xu, Hao Hao, kalyan kumar kalvagadda, > Nachiket Vaidya, and Vamsee Yarlagadda. > > > Bugs: SENTRY-1624 > https://issues.apache.org/jira/browse/SENTRY-1624 > > > Repository: sentry > > > Description > ------- > > SENTRY-1624 DefaultSentryValidator doesn't correctly construct > SentryOnFailureHookContextImpl > > > Diffs > ----- > > > sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/authorizer/DefaultSentryValidator.java > c9da3ab0a901095acc0d78ce2ed0da0f0038ee56 > > Diff: https://reviews.apache.org/r/56411/diff/ > > > Testing > ------- > > > Thanks, > > Alexander Kolbasov > >
