Github user pwendell commented on a diff in the pull request:

    https://github.com/apache/spark/pull/590#discussion_r12219136
  
    --- Diff: pom.xml ---
    @@ -213,6 +215,19 @@
             <groupId>org.eclipse.jetty</groupId>
             <artifactId>jetty-server</artifactId>
             <version>${jetty.version}</version>
    +        <exclusions>
    +          <exclusion>
    +            <groupId>org.eclipse.jetty.orbit</groupId>
    +            <artifactId>javax.servlet</artifactId>
    +          </exclusion>
    +        </exclusions>
    +      </dependency>
    +      <!-- SPARK-1693: Dependent on multiple versions of servlet-api jars 
lead to throw
    +       an SecurityException when Spark built for hadoop 2.3.0 , 2.4.0  -->
    +      <dependency>
    +        <groupId>javax.servlet</groupId>
    +        <artifactId>javax.servlet-api</artifactId>
    +        <version>3.0.1</version>
    --- End diff --
    
    This is a not a great idea, because for the older versions of Hadoop this 
will bump the `servlet-api` dependency from 2.X to 3.X...  
    
    It also looks like Hadoop officially advertises version 2.5 of the 
servlet-api for both 2.3.0 and 2.4.0:
    
http://repo1.maven.org/maven2/org/apache/hadoop/hadoop-project/2.3.0/hadoop-project-2.3.0.pom
    
http://repo1.maven.org/maven2/org/apache/hadoop/hadoop-project/2.4.0/hadoop-project-2.4.0.pom
    



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Reply via email to