> On Oct. 19, 2017, 7:25 p.m., kalyan kumar kalvagadda wrote: > > I'm fine with bumping up the jetty version but the only concern is that the > > Solr tests will be failing unitl changes for SENTRY-1475 are committed. > > Colm O hEigeartaigh wrote: > Agreed, probably best waiting until the Solr upgrade happens, assuming of > course that it's going to get done for 2.0.0?
I can sync with SENTRY-1475 and wait to commit this until a patch is available. The thing is that SENTRY-1475 needs Jetty9 to work, so we are in a chicken and egg problem. Unless we bump Jetty9 in the same SENTRY-1475 patch. Which one is better? - Sergio ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63160/#review188735 ----------------------------------------------------------- On Oct. 19, 2017, 7:17 p.m., Sergio Pena wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/63160/ > ----------------------------------------------------------- > > (Updated Oct. 19, 2017, 7:17 p.m.) > > > Review request for sentry. > > > Bugs: sentry-1667 > https://issues.apache.org/jira/browse/sentry-1667 > > > Repository: sentry > > > Description > ------- > > Bump Jetty to v9.4.7.v20170914 > > > Diffs > ----- > > pom.xml 1c055569e9176a1b1a7188b99dba5b383096bc20 > sentry-provider/sentry-provider-db/pom.xml > 4481e080d3477c1fe4410ca3f368a0c2cc3e501b > sentry-tests/sentry-tests-sqoop/pom.xml > 05b1653b1e46790a8ad7119e78b6d3d17baa4c6d > > > Diff: https://reviews.apache.org/r/63160/diff/1/ > > > Testing > ------- > > Only Solr tests are failing. But these should be addressed by SENTRY-1475 > where Solr 6 will be used. > > > Thanks, > > Sergio Pena > >
