Skip to content

Commit

Permalink
Use patched Ignite client
Browse files Browse the repository at this point in the history
  • Loading branch information
nineinchnick committed Aug 29, 2023
1 parent f662c6c commit f5c0b73
Showing 1 changed file with 37 additions and 6 deletions.
43 changes: 37 additions & 6 deletions plugin/trino-ignite/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -39,6 +39,18 @@
<artifactId>trino-base-jdbc</artifactId>
</dependency>

<!-- TODO Use org.apache.ignite:ignite-core:2.15.0 instead when new Ignite is released -->
<dependency>
<groupId>io.trino</groupId>
<artifactId>trino-ignite-patched</artifactId>
<exclusions>
<exclusion>
<groupId>org.apache.ignite</groupId>
<artifactId>*</artifactId>
</exclusion>
</exclusions>
</dependency>

<dependency>
<groupId>io.trino</groupId>
<artifactId>trino-matching</artifactId>
Expand All @@ -60,12 +72,6 @@
<artifactId>jakarta.validation-api</artifactId>
</dependency>

<dependency>
<groupId>org.apache.ignite</groupId>
<artifactId>ignite-core</artifactId>
<version>2.15.0</version>
</dependency>

<dependency>
<groupId>com.fasterxml.jackson.core</groupId>
<artifactId>jackson-annotations</artifactId>
Expand Down Expand Up @@ -214,4 +220,29 @@
<scope>test</scope>
</dependency>
</dependencies>

<!--
TODO convert this back into normal dependency
The Ignite code actually comes from trino-ignite-patched dependency, but IntelliJ does not support multi-module
projects with module using shading (as trino-ignite-patched does). See e.g. https://youtrack.jetbrains.com/issue/IDEA-266746
This dependency is here only to fool IntelliJ into compiling the project. -->
<profiles>
<profile>
<id>lib/tools.jar</id>
<activation>
<property>
<name>idea.maven.embedder.version</name>
</property>
</activation>
<dependencies>
<dependency>
<groupId>org.apache.ignite</groupId>
<artifactId>ignite-core</artifactId>
<version>2.15.0</version>
<scope>provided</scope>
<optional>true</optional>
</dependency>
</dependencies>
</profile>
</profiles>
</project>

0 comments on commit f5c0b73

Please sign in to comment.