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

    https://github.com/apache/spark/pull/15553#discussion_r84057228
  
    --- Diff: sql/hive-thriftserver/pom.xml ---
    @@ -41,11 +41,8 @@
           <version>${project.version}</version>
         </dependency>
         <dependency>
    -      <groupId>org.apache.spark</groupId>
    -      <artifactId>spark-core_${scala.binary.version}</artifactId>
    -      <version>${project.version}</version>
    -      <type>test-jar</type>
    -      <scope>test</scope>
    +      <groupId>org.eclipse.jetty</groupId>
    +      <artifactId>jetty-servlet</artifactId>
    --- End diff --
    
    Yeah, this was a unclear to me why it was required.
    I am reproducing the exact reported without this dependency added. I am 
hoping you might have a better idea why.
    The WebUI class does use jetty api; but that is in core - 
sql/hive/thriftserver/ui/ThriftServerPage.scala extends WebUIPage from core 
though - but there is no eclipse/jetty related dependency in thriftserver 
pom.xml
    
    ```
    [error] missing or invalid dependency detected while loading class file 
'WebUI.class'.
    [error] Could not access term eclipse in package org,
    [error] because it (or its dependencies) are missing. Check your build 
definition for
    [error] missing or conflicting dependencies. (Re-run with `-Ylog-classpath` 
to see the problematic classpath.)
    [error] A full rebuild may help if 'WebUI.class' was compiled against an 
incompatible version of org.
    [error] missing or invalid dependency detected while loading class file 
'WebUI.class'.
    [error] Could not access term jetty in value org.eclipse,
    [error] because it (or its dependencies) are missing. Check your build 
definition for
    [error] missing or conflicting dependencies. (Re-run with `-Ylog-classpath` 
to see the problematic classpath.)
    [error] A full rebuild may help if 'WebUI.class' was compiled against an 
incompatible version of org.eclipse.
    [error] two errors found
    ```



---
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.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

Reply via email to