----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53038/#review153408 -----------------------------------------------------------
pom.xml (line 510) <https://reviews.apache.org/r/53038/#comment222712> how about setting datanucleus-jdo.version in pom perperty and use ${datanucleus-jdo.version} here? sentry-binding/sentry-binding-hive-v2/pom.xml (line 35) <https://reviews.apache.org/r/53038/#comment222713> The change in line 146 is unnecessary, since datanucleus-jdo.version is already set to 3.2.0-m3 here. - Li Li On Oct. 20, 2016, 12:21 a.m., Alexander Kolbasov wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/53038/ > ----------------------------------------------------------- > > (Updated Oct. 20, 2016, 12:21 a.m.) > > > Review request for sentry, Colm O hEigeartaigh, Colin Ma, Hao Hao, Li Li, and > Sravya Tirukkovalur. > > > Bugs: SENTRY-1507 > https://issues.apache.org/jira/browse/SENTRY-1507 > > > Repository: sentry > > > Description > ------- > > SENTRY-1507 > > > Diffs > ----- > > pom.xml 4eb7d5f5814d65dfdf8cfda2e4ff0f964955f7c9 > sentry-binding/sentry-binding-hive-v2/pom.xml > e6df18ab70245f1b014a739cd81621357774c278 > sentry-service/sentry-service-server/pom.xml > 97bd326b0305303194a965a60b6afd0feefbbe0f > sentry-tests/sentry-tests-kafka/pom.xml > 4c0ae777f721315bb3e08081e20a4322abb45a72 > sentry-tests/sentry-tests-solr/pom.xml > a60b4eed4dde6c182b061681305013b3bd548cc3 > sentry-tests/sentry-tests-sqoop/pom.xml > f70c5c326a4cf3e98fab60dd9c74f8d2144ea4a3 > > Diff: https://reviews.apache.org/r/53038/diff/ > > > Testing > ------- > > > Thanks, > > Alexander Kolbasov > >