> On Jan. 4, 2017, 12:37 a.m., Vamsee Yarlagadda wrote: > > sentry-tests/sentry-tests-solr/pom.xml, line 58 > > <https://reviews.apache.org/r/55094/diff/2/?file=1596481#file1596481line58> > > > > It looks the server is using latest version of Jetty. > > <jettyVersion>8.1.19.v20160209</jettyVersion> > > > > And in the solr/sentry tests we are pulling down a old one. Will this > > conflict with the other version being run?
This is intentional. Tests run everything in a single JVM so later Jetty version conflicts wioth the Jetty version used by Solr. To work around this I added explicit older Jetty version in Solr sentry tests pom file. - Alexander ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55094/#review160457 ----------------------------------------------------------- On Jan. 4, 2017, 12:31 a.m., Alexander Kolbasov wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/55094/ > ----------------------------------------------------------- > > (Updated Jan. 4, 2017, 12:31 a.m.) > > > Review request for sentry, Colm O hEigeartaigh, Colin Ma, Hao Hao, kalyan > kumar kalvagadda, Vamsee Yarlagadda, and Vadim Spector. > > > Bugs: SENTRY-1532 > https://issues.apache.org/jira/browse/SENTRY-1532 > > > Repository: sentry > > > Description > ------- > > SENTRY-1532: Sentry Web UI isn't working > > > Diffs > ----- > > sentry-service/sentry-service-server/pom.xml > 97bd326b0305303194a965a60b6afd0feefbbe0f > sentry-tests/sentry-tests-solr/pom.xml > a60b4eed4dde6c182b061681305013b3bd548cc3 > > Diff: https://reviews.apache.org/r/55094/diff/ > > > Testing > ------- > > > Thanks, > > Alexander Kolbasov > >
