-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/63160/
-----------------------------------------------------------

(Updated Oct. 22, 2017, 3:09 a.m.)


Review request for sentry.


Changes
-------

Here's a better version of the Jetty9 patch.
- Add a Jetty8 dependency on the Solr tests to be able to pass.
- Modify the SentryWebServer so that it uses jetty9 API
- Exclude jetty7 dependencyes brought by Hive

There is no other Jetty version that is used in Sentry.


Bugs: sentry-1667
    https://issues.apache.org/jira/browse/sentry-1667


Repository: sentry


Description
-------

Bump Jetty to v9.4.7.v20170914


Diffs (updated)
-----

  pom.xml b1a04c4772b43e61855340e568c2268a2aa93a26 
  sentry-binding/sentry-binding-solr/pom.xml 
9cb8d27c0343019038fa19d011dbc0cf34e431f0 
  sentry-provider/sentry-provider-db/pom.xml 
4481e080d3477c1fe4410ca3f368a0c2cc3e501b 
  
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/thrift/SentryWebServer.java
 3f4a2ff7eb4e6655b516a457ba8f0329f41d0520 
  
sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/service/thrift/TestSentryWebServerWithKerberos.java
 09ee6b4493611c055dd7e96ab8a0b747fd4eb25b 
  sentry-tests/sentry-tests-solr/pom.xml 
efb708ce0ed62636b0377de3d49cd1bdaf6ff54c 
  sentry-tests/sentry-tests-sqoop/pom.xml 
05b1653b1e46790a8ad7119e78b6d3d17baa4c6d 


Diff: https://reviews.apache.org/r/63160/diff/2/

Changes: https://reviews.apache.org/r/63160/diff/1-2/


Testing
-------

Only Solr tests are failing. But these should be addressed by SENTRY-1475 where 
Solr 6 will be used.


Thanks,

Sergio Pena

Reply via email to