----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/20225/#review40058 -----------------------------------------------------------
Ship it! LGTM - Prasad Mujumdar On April 10, 2014, 6:45 p.m., Sravya Tirukkovalur wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/20225/ > ----------------------------------------------------------- > > (Updated April 10, 2014, 6:45 p.m.) > > > Review request for sentry, Jarek Cecho and Prasad Mujumdar. > > > Bugs: SENTRY-168 > https://issues.apache.org/jira/browse/SENTRY-168 > > > Repository: sentry > > > Description > ------- > > Minor fixes to README and pom.xml: > > * We cannot use mvn compile to compile Sentry because of this issue: > http://jira.codehaus.org/browse/MEXEC-91 > * We need wget to run hive end to end tests > * Add descriptions for the rat excludes > > > Diffs > ----- > > README.md 2e142cf4f72ea806f5a03f04df447e18fb2f3b5c > pom.xml 91cc88aa57613e39ce727f6bf68b5c5c128f3a09 > > Diff: https://reviews.apache.org/r/20225/diff/ > > > Testing > ------- > > rm -rf ~/.m2/* > mvn clean install -DskipTests > mvn verify > > > Thanks, > > Sravya Tirukkovalur > >
