You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@beam.apache.org by "Abacn (via GitHub)" <gi...@apache.org> on 2024/10/22 20:40:05 UTC
Re: [PR] Bump zetasql version to 2024.08.1 [beam]
Abacn commented on PR #32902:
URL: https://github.com/apache/beam/pull/32902#issuecomment-2430220462
ZetaSQL 2022.08 release already had multiple breaking change due to removed public APIs:
```
Task :sdks:java:extensions:sql:hcatalog:buildDependents
/runner/_work/beam/beam/sdks/java/extensions/sql/zetasql/src/main/java/org/apache/beam/sdk/extensions/sql/zetasql/translation/ExpressionConverter.java:500: error: cannot find symbol
> Task :sdks:java:extensions:sql:zetasql:compileJava
fixedOutputSchemaTVF.getOutputSchema().getColumns());
^
symbol: method getOutputSchema()
location: variable fixedOutputSchemaTVF of type FixedOutputSchemaTVF
/runner/_work/beam/beam/sdks/java/extensions/sql/zetasql/src/main/java/org/apache/beam/sdk/extensions/sql/zetasql/translation/TVFScanConverter.java:74: error: cannot find symbol
.containsKey(zetaNode.getTvf().getNamePath())) {
^
symbol: method getNamePath()
location: class TableValuedFunction
/runner/_work/beam/beam/sdks/java/extensions/sql/zetasql/src/main/java/org/apache/beam/sdk/extensions/sql/zetasql/translation/TVFScanConverter.java:75: error: cannot find symbol
inputs.add(context.getUserDefinedTableValuedFunctions().get(zetaNode.getTvf().getNamePath()));
^
symbol: method getNamePath()
location: class TableValuedFunction
/runner/_work/beam/beam/sdks/java/extensions/sql/zetasql/src/main/java/org/apache/beam/sdk/extensions/sql/zetasql/translation/AggregateScanConverter.java:97: error: incompatible types: ResolvedComputedColumnBase cannot be converted to ResolvedComputedColumn
for (ResolvedComputedColumn computedColumn : zetaNode.getAggregateList()) {
^
/runner/_work/beam/beam/sdks/java/extensions/sql/zetasql/src/main/java/org/apache/beam/sdk/extensions/sql/zetasql/translation/AggregateScanConverter.java:147: error: incompatible types: ResolvedComputedColumnBase cannot be converted to ResolvedComputedColumn
for (ResolvedComputedColumn resolvedComputedColumn : node.getAggregateList()) {
^
```
Found internally that these code were removed by bot as "dead code". Upgrading ZetaSQL is no longer not attempted as more similar breaking change may be seen if we choose this route.
--
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: github-unsubscribe@beam.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org