[
https://issues.apache.org/jira/browse/DRILL-7135?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17352031#comment-17352031
]
ASF GitHub Bot commented on DRILL-7135:
---------------------------------------
paul-rogers commented on a change in pull request #2236:
URL: https://github.com/apache/drill/pull/2236#discussion_r640055304
##########
File path: exec/java-exec/pom.xml
##########
@@ -502,6 +502,11 @@
</exclusion>
</exclusions>
</dependency>
+ <dependency>
+ <groupId>org.apache.hadoop</groupId>
+ <artifactId>hadoop-yarn-api</artifactId>
+ <scope>test</scope>
+ </dependency>
Review comment:
Is this needed here? Drill-on-YARN uses the YARN API. We did try hard to
avoid adding a YARN dependency to Drill itself, since Drill can run
stand-alone, under YARN, under K8s, etc.
##########
File path: pom.xml
##########
@@ -2850,43 +2850,10 @@
<!--Eclipse Jetty dependecies-->
<dependency>
<groupId>org.eclipse.jetty</groupId>
- <artifactId>jetty-server</artifactId>
- <version>${jetty.version}</version>
- </dependency>
- <dependency>
- <groupId>org.eclipse.jetty</groupId>
- <artifactId>jetty-servlet</artifactId>
- <version>${jetty.version}</version>
- </dependency>
- <dependency>
- <groupId>org.eclipse.jetty</groupId>
- <artifactId>jetty-servlets</artifactId>
- <version>${jetty.version}</version>
- </dependency>
- <dependency>
- <groupId>org.eclipse.jetty</groupId>
- <artifactId>jetty-security</artifactId>
- <version>${jetty.version}</version>
- </dependency>
- <dependency>
- <groupId>org.eclipse.jetty</groupId>
- <artifactId>jetty-util</artifactId>
- <version>${jetty.version}</version>
- </dependency>
- <dependency>
- <groupId>org.eclipse.jetty</groupId>
- <artifactId>jetty-io</artifactId>
- <version>${jetty.version}</version>
- </dependency>
- <dependency>
- <groupId>org.eclipse.jetty</groupId>
- <artifactId>jetty-webapp</artifactId>
- <version>${jetty.version}</version>
- </dependency>
- <dependency>
- <groupId>org.eclipse.jetty</groupId>
- <artifactId>jetty-xml</artifactId>
+ <artifactId>jetty-bom</artifactId>
<version>${jetty.version}</version>
+ <type>pom</type>
+ <scope>import</scope>
Review comment:
Nice simplification!
##########
File path:
drill-yarn/src/main/java/org/apache/drill/yarn/appMaster/http/WebServer.java
##########
@@ -306,18 +304,18 @@ private ConstraintSecurityHandler createSecurityHandler()
{
}
/**
- * It creates A {@link SessionHandler} which contains a {@link
HashSessionManager}
+ * It creates A {@link SessionHandler}
Review comment:
Sorry to make you do this twice. The DoY web server was pretty much a
copy/paste clone of the Drill exec version. The goal was to factor out a common
set of code, but that never quite happened...
--
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:
[email protected]
> Upgrade to Jetty 9.4
> --------------------
>
> Key: DRILL-7135
> URL: https://issues.apache.org/jira/browse/DRILL-7135
> Project: Apache Drill
> Issue Type: Improvement
> Affects Versions: 1.15.0
> Reporter: Vitalii Diravka
> Priority: Minor
>
> Initially DRILL-7051 updated Jetty to 9.4 version and DRILL-7081 updated
> Jersey version to 2.28 version. These versions work fine for Drill with
> Hadoop version below 3.0.
> Starting from Hadoop 3.0 it uses
> [org.eclipse.jetty|https://github.com/apache/hadoop/blob/branch-3.0/hadoop-project/pom.xml#L38]
> 9.3 version.
> That's why it conflicts with newer Jetty&Jersey versions.
> Drill can update Jetty and Jersey versions after resolution HADOOP-14930 and
> HBASE-19256.
> Or alternatively these libs can be shaded in Drill, but there is no real
> reason to do it nowadays.
> See details in
> [#1681|https://github.com/apache/drill/pull/1681#discussion_r265904521] PR.
> _Notes_:
> * For Jersey update it is necessary to add
> org.glassfish.jersey.inject:jersey-hk2 in Drill to solve all compilation
> failures.
> * See doc for Jetty update:
> https://www.eclipse.org/jetty/documentation/9.4.x/upgrading-jetty.html
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
