nielsbasjes commented on a change in pull request #12962:
URL: https://github.com/apache/beam/pull/12962#discussion_r497365708



##########
File path: vendor/calcite-1_25_0/build.gradle
##########
@@ -18,29 +18,33 @@
 
 plugins { id 'org.apache.beam.vendor-java' }
 
-description = "Apache Beam :: Vendored Dependencies :: Calcite 1.20.0"
+description = "Apache Beam :: Vendored Dependencies :: Calcite 1.25.0"
 
 group = "org.apache.beam"
 version = "0.2"
 
-def calcite_version = "1.20.0"
+def calcite_version = "1.25.0"
 def avatica_version = "1.16.0"
-def prefix = "org.apache.beam.vendor.calcite.v1_20_0"
+def prefix = "org.apache.beam.vendor.calcite.v1_25_0"
 
 List<String> packagesToRelocate = [
         "com.esri",
+        "com.fasterxml",
         "com.google.common",
-        "com.google.thirdparty",
         "com.google.protobuf",
-        "com.fasterxml",
+        "com.google.thirdparty",
         "com.jayway",
         "com.yahoo",
+        "javax.annotation",
+        "net.minidev",
         "org.apache.calcite",
         "org.apache.commons",
         "org.apache.http",
+        "org.apiguardian.api",
         "org.codehaus",
+        "org.objectweb",
         "org.pentaho",
-        "org.yaml"
+        "org.yaml",
 ]
 
 vendorJava(

Review comment:
       I have copied (+ updated version) of the runtime dependencies part of 
this.
   As this is uncharted territory for me I'm unsure if this is correct now.




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to