[GitHub] [beam] kennknowles commented on pull request #12962: [WIP][BEAM-9379] Update calcite to 1.25

2020-09-30 Thread GitBox
kennknowles commented on pull request #12962: URL: https://github.com/apache/beam/pull/12962#issuecomment-701704059 https://scans.gradle.com/s/ofcfnu774du4y/tests/:sdks:java:extensions:sql:test/org.apache.beam.sdk.extensions.sql.impl.udf.BeamSqlUdfExpressionTest/testSHA1?anchor=e30=0-1-0#1

[GitHub] [beam] kennknowles commented on pull request #12962: [WIP][BEAM-9379] Update calcite to 1.25

2020-09-30 Thread GitBox
kennknowles commented on pull request #12962: URL: https://github.com/apache/beam/pull/12962#issuecomment-701703858 https://scans.gradle.com/s/ofcfnu774du4y/tests/:sdks:java:extensions:sql:test/org.apache.beam.sdk.extensions.sql.BeamSqlDslAggregationTest/testWindowOnNonTimestampField#1

[GitHub] [beam] kennknowles commented on pull request #12962: [WIP][BEAM-9379] Update calcite to 1.25

2020-09-30 Thread GitBox
kennknowles commented on pull request #12962: URL: https://github.com/apache/beam/pull/12962#issuecomment-701703585 https://scans.gradle.com/s/ofcfnu774du4y/tests/:sdks:java:extensions:sql:test/org.apache.beam.sdk.extensions.sql.impl.rel.BeamMatchRelTest/matchPrevFunctionTest#1

[GitHub] [beam] kennknowles commented on pull request #12962: [WIP][BEAM-9379] Update calcite to 1.25

2020-09-30 Thread GitBox
kennknowles commented on pull request #12962: URL: https://github.com/apache/beam/pull/12962#issuecomment-701703218 https://scans.gradle.com/s/ofcfnu774du4y/tests/:sdks:java:extensions:sql:zetasql:test/org.apache.beam.sdk.extensions.sql.zetasql.ZetaSqlTimeFunctionsTest/testFormatDate#1

[GitHub] [beam] kennknowles commented on pull request #12962: [WIP][BEAM-9379] Update calcite to 1.25

2020-09-30 Thread GitBox
kennknowles commented on pull request #12962: URL: https://github.com/apache/beam/pull/12962#issuecomment-701702697 https://scans.gradle.com/s/ofcfnu774du4y/tests/:sdks:java:extensions:sql:test/org.apache.beam.sdk.extensions.sql.BeamSqlDslUdfUdafTest/testUdfWithListInput#1 Seems

[GitHub] [beam] kennknowles commented on pull request #12962: [WIP][BEAM-9379] Update calcite to 1.25

2020-09-30 Thread GitBox
kennknowles commented on pull request #12962: URL: https://github.com/apache/beam/pull/12962#issuecomment-701702361 https://scans.gradle.com/s/ofcfnu774du4y/tests/:sdks:java:extensions:sql:test/org.apache.beam.sdk.extensions.sql.BeamSqlDslSqlStdOperatorsTest/testArithmeticOperator#1

[GitHub] [beam] kennknowles commented on pull request #12962: [WIP][BEAM-9379] Update calcite to 1.25

2020-09-29 Thread GitBox
kennknowles commented on pull request #12962: URL: https://github.com/apache/beam/pull/12962#issuecomment-700894541 It might be a huge amount of work. We can just add the new vendored module and then release it, but we will want to know that we have enough people available to also port to

[GitHub] [beam] kennknowles commented on pull request #12962: [WIP][BEAM-9379] Update calcite to 1.25

2020-09-29 Thread GitBox
kennknowles commented on pull request #12962: URL: https://github.com/apache/beam/pull/12962#issuecomment-700893568 Can you publish a build scan by running your second step with `--scan` ? Then we can see what is broken.