[GitHub] [flink] snuyanzin commented on pull request #21203: [FLINK-21239][Table SQL/API] Upgrade Calcite to 1.28.0

2022-11-15 Thread GitBox
snuyanzin commented on PR #21203: URL: https://github.com/apache/flink/pull/21203#issuecomment-1315661164 @flinkbot run azure -- 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

[GitHub] [flink] snuyanzin commented on pull request #21203: [FLINK-21239][Table SQL/API] Upgrade Calcite to 1.28.0

2022-11-14 Thread GitBox
snuyanzin commented on PR #21203: URL: https://github.com/apache/flink/pull/21203#issuecomment-1314904648 Current failure is related to https://issues.apache.org/jira/browse/FLINK-24119 There are some fixes however this branch does not include them since it is not rebased... -- This

[GitHub] [flink] snuyanzin commented on pull request #21203: [FLINK-21239][Table SQL/API] Upgrade Calcite to 1.28.0

2022-11-14 Thread GitBox
snuyanzin commented on PR #21203: URL: https://github.com/apache/flink/pull/21203#issuecomment-1314873046 @flinkbot run azure -- 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

[GitHub] [flink] snuyanzin commented on pull request #21203: [FLINK-21239][Table SQL/API] Upgrade Calcite to 1.28.0

2022-11-14 Thread GitBox
snuyanzin commented on PR #21203: URL: https://github.com/apache/flink/pull/21203#issuecomment-1314832541 @flinkbot run azure -- 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

[GitHub] [flink] snuyanzin commented on pull request #21203: [FLINK-21239][Table SQL/API] Upgrade Calcite to 1.28.0

2022-11-14 Thread GitBox
snuyanzin commented on PR #21203: URL: https://github.com/apache/flink/pull/21203#issuecomment-1314550031 @twalthr thanks for your feedback I adapted PR to use Janino 3.0.11 (however it requires to copy/paste and change 4 Calcite classes which could be removed once Janino is upgraded)

[GitHub] [flink] snuyanzin commented on pull request #21203: [FLINK-21239][Table SQL/API] Upgrade Calcite to 1.28.0

2022-10-31 Thread GitBox
snuyanzin commented on PR #21203: URL: https://github.com/apache/flink/pull/21203#issuecomment-1297391537 @flinkbot run azure -- 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