vdiravka commented on a change in pull request #1681: DRILL-7051: Upgrade jetty
URL: https://github.com/apache/drill/pull/1681#discussion_r265904521
##########
File path: pom.xml
##########
@@ -85,6 +85,7 @@
<reflections.version>0.9.10</reflections.version>
<avro.version>1.8.2</avro.version>
<metrics.version>4.0.2</metrics.version>
+ <jetty.version>9.4.15.v20190215</jetty.version>
Review comment:
It looks like HBase was earlier on Jetty 9.4 version, but there is an issue
with `MiniDFSCluster`
[HBASE-18943](https://issues.apache.org/jira/browse/HBASE-18943),
[HBASE-19390](https://issues.apache.org/jira/browse/HBASE-19390), so they
reverted to 9.3 Jetty version. It was the same reason why I have updated Jetty
in the scope of [DRILL-6540: Upgrade to HADOOP-3.1
libraries](https://issues.apache.org/jira/browse/DRILL-6540).
It also appears that new Jersey version requires Jetty 9.4 version, see
[comment](https://github.com/apache/hbase/blob/master/pom.xml#L1533) in HBase.
Possibly we should consider the newest Jetty and Jersey versions only after
[HBASE-19256](https://issues.apache.org/jira/browse/HBASE-19256) resolution.
So I will check my changes for Hadoop3.1 libs version in Drill with newest
Jetty and Jersey. If there are some issues with them, I will put to Drill the
same Jetty and Jersey versions as in HBase project.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
With regards,
Apache Git Services
