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