MartijnVisser commented on pull request #17728: URL: https://github.com/apache/flink/pull/17728#issuecomment-975352496
To be sure we're on the same page, this is the current dependency tree on master: ``` [INFO] org.apache.flink:flink-sql-parser:jar:1.15-SNAPSHOT [INFO] +- org.apache.flink:flink-annotations:jar:1.15-SNAPSHOT:compile [INFO] +- com.google.guava:guava:jar:29.0-jre:provided [INFO] | +- com.google.guava:failureaccess:jar:1.0.1:provided [INFO] | +- com.google.guava:listenablefuture:jar:9999.0-empty-to-avoid-conflict-with-guava:provided [INFO] | +- org.checkerframework:checker-qual:jar:2.11.1:provided [INFO] | +- com.google.errorprone:error_prone_annotations:jar:2.3.4:provided [INFO] | \- com.google.j2objc:j2objc-annotations:jar:1.3:provided [INFO] +- org.apache.calcite:calcite-core:jar:1.26.0:compile [INFO] | +- org.apache.calcite.avatica:avatica-core:jar:1.17.0:compile [INFO] | \- org.apiguardian:apiguardian-api:jar:1.1.0:compile [INFO] +- org.apache.calcite:calcite-core:test-jar:tests:1.26.0:test [INFO] | +- org.apache.calcite:calcite-linq4j:jar:1.26.0:test [INFO] | +- com.esri.geometry:esri-geometry-api:jar:2.2.0:test [INFO] | +- com.fasterxml.jackson.dataformat:jackson-dataformat-yaml:jar:2.13.0:test [INFO] | | \- org.yaml:snakeyaml:jar:1.27:test [INFO] | +- com.jayway.jsonpath:json-path:jar:2.4.0:test [INFO] | | \- net.minidev:json-smart:jar:2.3:test [INFO] | | \- net.minidev:accessors-smart:jar:1.2:test [INFO] | | \- org.ow2.asm:asm:jar:5.0.4:test [INFO] | +- com.yahoo.datasketches:sketches-core:jar:0.9.0:test [INFO] | | \- com.yahoo.datasketches:memory:jar:0.9.0:test [INFO] | +- commons-codec:commons-codec:jar:1.15:test [INFO] | +- net.hydromatic:aggdesigner-algorithm:jar:6.0:test [INFO] | | +- commons-lang:commons-lang:jar:2.4:test [INFO] | | \- commons-logging:commons-logging:jar:1.1.3:test [INFO] | +- org.apache.commons:commons-dbcp2:jar:2.6.0:test [INFO] | | \- org.apache.commons:commons-pool2:jar:2.6.1:test [INFO] | +- org.apache.commons:commons-lang3:jar:3.3.2:test [INFO] | +- commons-io:commons-io:jar:2.11.0:test [INFO] | +- org.codehaus.janino:commons-compiler:jar:3.0.11:test [INFO] | \- org.codehaus.janino:janino:jar:3.0.11:test [INFO] +- org.apache.flink:flink-shaded-force-shading:jar:14.0:compile [INFO] +- org.slf4j:slf4j-api:jar:1.7.15:compile [INFO] +- com.google.code.findbugs:jsr305:jar:1.3.9:compile [INFO] +- org.junit.jupiter:junit-jupiter:jar:5.7.2:test [INFO] | +- org.junit.jupiter:junit-jupiter-api:jar:5.7.2:test [INFO] | | +- org.opentest4j:opentest4j:jar:1.2.0:test [INFO] | | \- org.junit.platform:junit-platform-commons:jar:1.7.2:test [INFO] | +- org.junit.jupiter:junit-jupiter-params:jar:5.7.2:test [INFO] | \- org.junit.jupiter:junit-jupiter-engine:jar:5.7.2:test [INFO] +- org.junit.vintage:junit-vintage-engine:jar:5.7.2:test [INFO] | +- org.junit.platform:junit-platform-engine:jar:1.7.2:test [INFO] | \- junit:junit:jar:4.13.2:test [INFO] +- org.mockito:mockito-core:jar:2.21.0:test [INFO] | +- net.bytebuddy:byte-buddy:jar:1.8.15:test [INFO] | +- net.bytebuddy:byte-buddy-agent:jar:1.8.15:test [INFO] | \- org.objenesis:objenesis:jar:2.1:test [INFO] +- org.powermock:powermock-module-junit4:jar:2.0.9:test [INFO] | +- org.powermock:powermock-module-junit4-common:jar:2.0.9:test [INFO] | | +- org.powermock:powermock-reflect:jar:2.0.9:test [INFO] | | \- org.powermock:powermock-core:jar:2.0.9:test [INFO] | | \- org.javassist:javassist:jar:3.24.0-GA:test [INFO] | \- org.hamcrest:hamcrest-core:jar:1.3:test [INFO] +- org.powermock:powermock-api-mockito2:jar:2.0.9:test [INFO] | \- org.powermock:powermock-api-support:jar:2.0.9:test [INFO] +- org.hamcrest:hamcrest-all:jar:1.3:test [INFO] +- org.testcontainers:junit-jupiter:jar:1.16.2:test [INFO] | \- org.testcontainers:testcontainers:jar:1.16.2:test [INFO] | +- org.apache.commons:commons-compress:jar:1.21:test [INFO] | +- org.rnorth.duct-tape:duct-tape:jar:1.0.8:test [INFO] | | \- org.jetbrains:annotations:jar:17.0.0:test [INFO] | +- com.github.docker-java:docker-java-api:jar:3.2.12:test [INFO] | | \- com.fasterxml.jackson.core:jackson-annotations:jar:2.13.0:test [INFO] | \- com.github.docker-java:docker-java-transport-zerodep:jar:3.2.12:test [INFO] | +- com.github.docker-java:docker-java-transport:jar:3.2.12:test [INFO] | \- net.java.dev.jna:jna:jar:5.8.0:test [INFO] +- org.apache.logging.log4j:log4j-slf4j-impl:jar:2.14.1:test [INFO] +- org.apache.logging.log4j:log4j-api:jar:2.14.1:test [INFO] +- org.apache.logging.log4j:log4j-core:jar:2.14.1:test [INFO] \- org.apache.logging.log4j:log4j-1.2-api:jar:2.14.1:test ``` As you can see, `org.apiguardian:apiguardian-api:jar:1.1.0:compile` is there. In this PR, the dependency tree looks like this: ``` [INFO] org.apache.flink:flink-sql-parser:jar:1.15-SNAPSHOT [INFO] +- org.apache.flink:flink-annotations:jar:1.15-SNAPSHOT:compile [INFO] +- com.google.guava:guava:jar:29.0-jre:provided [INFO] | +- com.google.guava:failureaccess:jar:1.0.1:provided [INFO] | +- com.google.guava:listenablefuture:jar:9999.0-empty-to-avoid-conflict-with-guava:provided [INFO] | +- org.checkerframework:checker-qual:jar:2.11.1:provided [INFO] | +- com.google.errorprone:error_prone_annotations:jar:2.3.4:provided [INFO] | \- com.google.j2objc:j2objc-annotations:jar:1.3:provided [INFO] +- org.apache.calcite:calcite-core:jar:1.26.0:compile [INFO] | \- org.apache.calcite.avatica:avatica-core:jar:1.17.0:compile [INFO] +- org.apache.calcite:calcite-core:test-jar:tests:1.26.0:test [INFO] | +- org.apache.calcite:calcite-linq4j:jar:1.26.0:test [INFO] | +- com.esri.geometry:esri-geometry-api:jar:2.2.0:test [INFO] | +- com.fasterxml.jackson.dataformat:jackson-dataformat-yaml:jar:2.13.0:test [INFO] | | \- org.yaml:snakeyaml:jar:1.27:test [INFO] | +- com.jayway.jsonpath:json-path:jar:2.4.0:test [INFO] | | \- net.minidev:json-smart:jar:2.3:test [INFO] | | \- net.minidev:accessors-smart:jar:1.2:test [INFO] | | \- org.ow2.asm:asm:jar:5.0.4:test [INFO] | +- com.yahoo.datasketches:sketches-core:jar:0.9.0:test [INFO] | | \- com.yahoo.datasketches:memory:jar:0.9.0:test [INFO] | +- commons-codec:commons-codec:jar:1.15:test [INFO] | +- net.hydromatic:aggdesigner-algorithm:jar:6.0:test [INFO] | | +- commons-lang:commons-lang:jar:2.4:test [INFO] | | \- commons-logging:commons-logging:jar:1.1.3:test [INFO] | +- org.apache.commons:commons-dbcp2:jar:2.6.0:test [INFO] | | \- org.apache.commons:commons-pool2:jar:2.6.1:test [INFO] | +- org.apache.commons:commons-lang3:jar:3.3.2:test [INFO] | +- commons-io:commons-io:jar:2.11.0:test [INFO] | +- org.codehaus.janino:commons-compiler:jar:3.0.11:test [INFO] | \- org.codehaus.janino:janino:jar:3.0.11:test [INFO] +- org.apache.flink:flink-shaded-force-shading:jar:14.0:compile [INFO] +- org.slf4j:slf4j-api:jar:1.7.15:compile [INFO] +- com.google.code.findbugs:jsr305:jar:1.3.9:compile [INFO] +- org.junit.jupiter:junit-jupiter:jar:5.8.1:test [INFO] | +- org.junit.jupiter:junit-jupiter-api:jar:5.8.1:test [INFO] | | +- org.opentest4j:opentest4j:jar:1.2.0:test [INFO] | | \- org.junit.platform:junit-platform-commons:jar:1.8.1:test [INFO] | +- org.junit.jupiter:junit-jupiter-params:jar:5.8.1:test [INFO] | \- org.junit.jupiter:junit-jupiter-engine:jar:5.8.1:test [INFO] +- org.junit.vintage:junit-vintage-engine:jar:5.8.1:test [INFO] | +- org.junit.platform:junit-platform-engine:jar:1.8.1:test [INFO] | +- junit:junit:jar:4.13.2:test [INFO] | \- org.apiguardian:apiguardian-api:jar:1.1.2:test [INFO] +- org.mockito:mockito-core:jar:2.21.0:test [INFO] | +- net.bytebuddy:byte-buddy:jar:1.8.15:test [INFO] | +- net.bytebuddy:byte-buddy-agent:jar:1.8.15:test [INFO] | \- org.objenesis:objenesis:jar:2.1:test [INFO] +- org.powermock:powermock-module-junit4:jar:2.0.4:test [INFO] | +- org.powermock:powermock-module-junit4-common:jar:2.0.4:test [INFO] | | +- org.powermock:powermock-reflect:jar:2.0.4:test [INFO] | | \- org.powermock:powermock-core:jar:2.0.4:test [INFO] | | \- org.javassist:javassist:jar:3.24.0-GA:test [INFO] | \- org.hamcrest:hamcrest-core:jar:1.3:test [INFO] +- org.powermock:powermock-api-mockito2:jar:2.0.4:test [INFO] | \- org.powermock:powermock-api-support:jar:2.0.4:test [INFO] +- org.hamcrest:hamcrest-all:jar:1.3:test [INFO] +- org.testcontainers:junit-jupiter:jar:1.16.2:test [INFO] | \- org.testcontainers:testcontainers:jar:1.16.2:test [INFO] | +- org.apache.commons:commons-compress:jar:1.21:test [INFO] | +- org.rnorth.duct-tape:duct-tape:jar:1.0.8:test [INFO] | | \- org.jetbrains:annotations:jar:17.0.0:test [INFO] | +- com.github.docker-java:docker-java-api:jar:3.2.12:test [INFO] | | \- com.fasterxml.jackson.core:jackson-annotations:jar:2.13.0:test [INFO] | \- com.github.docker-java:docker-java-transport-zerodep:jar:3.2.12:test [INFO] | +- com.github.docker-java:docker-java-transport:jar:3.2.12:test [INFO] | \- net.java.dev.jna:jna:jar:5.8.0:test [INFO] +- org.apache.logging.log4j:log4j-slf4j-impl:jar:2.14.1:test [INFO] +- org.apache.logging.log4j:log4j-api:jar:2.14.1:test [INFO] +- org.apache.logging.log4j:log4j-core:jar:2.14.1:test [INFO] \- org.apache.logging.log4j:log4j-1.2-api:jar:2.14.1:test ``` So there is `org.apiguardian:apiguardian-api:jar:1.1.2:test` as a test dependency, while Calcite is still there in the JAR. I've build it locally and ran the `WordCountSQLExample.jar` from `/examples/table`. LMKWYT @matriv @slinkydeveloper -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
