This is an automated email from the ASF dual-hosted git repository.

vlsi pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/calcite.git


The following commit(s) were added to refs/heads/main by this push:
     new 76c2f5fe04 [CALCITE-4949] Evaluate if using org.codelibs is 
appropriate for an Elasticsearch painless dependency
76c2f5fe04 is described below

commit 76c2f5fe0416db42673be0df75b6df4d56cd291c
Author: Zhe Hu <[email protected]>
AuthorDate: Sat Dec 18 13:00:08 2021 +0800

    [CALCITE-4949] Evaluate if using org.codelibs is appropriate for an 
Elasticsearch painless dependency
---
 bom/build.gradle.kts           | 1 -
 elasticsearch/build.gradle.kts | 6 +++++-
 2 files changed, 5 insertions(+), 2 deletions(-)

diff --git a/bom/build.gradle.kts b/bom/build.gradle.kts
index 64cb532afd..5febf6f0b3 100644
--- a/bom/build.gradle.kts
+++ b/bom/build.gradle.kts
@@ -130,7 +130,6 @@ fun DependencyConstraintHandlerScope.runtimev(
         apiv("org.codehaus.janino:commons-compiler", "janino")
         apiv("org.codehaus.janino:janino")
         apiv("org.codelibs.elasticsearch.module:lang-painless", 
"elasticsearch")
-        apiv("org.codelibs.elasticsearch.module:scripting-painless-spi", 
"elasticsearch")
         apiv("org.eclipse.jetty:jetty-http", "jetty")
         apiv("org.eclipse.jetty:jetty-security", "jetty")
         apiv("org.eclipse.jetty:jetty-server", "jetty")
diff --git a/elasticsearch/build.gradle.kts b/elasticsearch/build.gradle.kts
index 0ce6db032d..057db6bc9f 100644
--- a/elasticsearch/build.gradle.kts
+++ b/elasticsearch/build.gradle.kts
@@ -37,7 +37,11 @@
 
     testImplementation("org.apache.logging.log4j:log4j-api")
     testImplementation("org.apache.logging.log4j:log4j-core")
-    testImplementation("org.codelibs.elasticsearch.module:lang-painless")
+    // See https://github.com/elastic/elasticsearch/issues/18131.
+    testImplementation("org.codelibs.elasticsearch.module:lang-painless") {
+        because("Elasticsearch doesn't export painless script artifact to 
maven central, " +
+                "so using 3rd party version (codelibs) only for test.")
+    }
     testImplementation("org.elasticsearch.plugin:transport-netty4-client")
     testImplementation("org.elasticsearch:elasticsearch")
     testImplementation(project(":testkit"))

Reply via email to