You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@orc.apache.org by wi...@apache.org on 2021/08/06 04:24:57 UTC
[orc] branch main updated: ORC-911: Remove janino dependency in
favor of Spark's transitive dependency (#817)
This is an automated email from the ASF dual-hosted git repository.
william pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/orc.git
The following commit(s) were added to refs/heads/main by this push:
new 5c6008d ORC-911: Remove janino dependency in favor of Spark's transitive dependency (#817)
5c6008d is described below
commit 5c6008d1ad13b8dde39c427efd39eaff934ee773
Author: Dongjoon Hyun <do...@apache.org>
AuthorDate: Thu Aug 5 21:24:51 2021 -0700
ORC-911: Remove janino dependency in favor of Spark's transitive dependency (#817)
### What changes were proposed in this pull request?
This PR aims to remove `janino` runtime dependency from `bench` module in favor of Spark's transitive dependency.
### Why are the changes needed?
We should not override it. Spark uses a specific version because some latest Janino has bugs.
### How was this patch tested?
Pass the CIs.
Closes #799 and #804 .
---
java/bench/pom.xml | 12 ------------
java/bench/spark/pom.xml | 8 --------
2 files changed, 20 deletions(-)
diff --git a/java/bench/pom.xml b/java/bench/pom.xml
index 2da11a2..a383c3d 100644
--- a/java/bench/pom.xml
+++ b/java/bench/pom.xml
@@ -409,18 +409,6 @@
<version>${spark.version}</version>
</dependency>
<dependency>
- <groupId>org.codehaus.janino</groupId>
- <artifactId>janino</artifactId>
- <version>3.0.8</version>
- <scope>runtime</scope>
- </dependency>
- <dependency>
- <groupId>org.codehaus.janino</groupId>
- <artifactId>commons-compiler</artifactId>
- <version>3.0.8</version>
- <scope>runtime</scope>
- </dependency>
- <dependency>
<groupId>org.jodd</groupId>
<artifactId>jodd-core</artifactId>
<version>3.5.2</version>
diff --git a/java/bench/spark/pom.xml b/java/bench/spark/pom.xml
index 1093da1..02e94b6 100644
--- a/java/bench/spark/pom.xml
+++ b/java/bench/spark/pom.xml
@@ -111,14 +111,6 @@
<artifactId>spark-avro_2.12</artifactId>
</dependency>
<dependency>
- <groupId>org.codehaus.janino</groupId>
- <artifactId>janino</artifactId>
- </dependency>
- <dependency>
- <groupId>org.codehaus.janino</groupId>
- <artifactId>commons-compiler</artifactId>
- </dependency>
- <dependency>
<groupId>org.jodd</groupId>
<artifactId>jodd-core</artifactId>
</dependency>