Skip to content

Commit

Permalink
IDEA-93855
Browse files Browse the repository at this point in the history
  • Loading branch information
pan3793 committed Sep 18, 2024
1 parent e8eb156 commit a12b76b
Showing 1 changed file with 12 additions and 8 deletions.
20 changes: 12 additions & 8 deletions kyuubi-server/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -77,7 +77,7 @@

<dependency>
<groupId>org.apache.kyuubi</groupId>
<artifactId>kyuubi-hive-jdbc</artifactId>
<artifactId>${hive.jdbc.artifact}</artifactId>
<version>${project.version}</version>
</dependency>

Expand Down Expand Up @@ -269,17 +269,21 @@
<scope>test</scope>
</dependency>

<dependency>
<groupId>org.apache.kyuubi</groupId>
<artifactId>${hive.jdbc.artifact}</artifactId>
<version>${project.version}</version>
<scope>test</scope>
</dependency>

<!--
The server module does not consume the spark-sql-engine classes, but it supposes that
spark-sql-engine's jar existed on the target folder, so here we declare it as a dependency
to make sure that Maven always process spark-sql-engine module before the server module.
IntelliJ IDEA 2024.1 fixed the IDEA-93855, thus the relocated classes inside the
spark-sql-engine's shaded jar are visible in the server module in IDEA, for example,
`org.apache.kyuubi.shaded.spark.connect.proto.ExecutePlanRequest`, which silently breaks
the IDEA code analysis and jumping capabilities. This seems could be workaround by changing
the dependency type from `jar`(default value) to `pom`.
-->
<dependency>
<groupId>org.apache.kyuubi</groupId>
<artifactId>kyuubi-spark-sql-engine_${scala.binary.version}</artifactId>
<version>${project.version}</version>
<type>pom</type>
<scope>test</scope>
</dependency>

Expand Down

0 comments on commit a12b76b

Please sign in to comment.