[GitHub] [calcite] JiajunBernoulli commented on a diff in pull request #2968: [CALCITE-5354] JDBC with UNNEST not working

2022-12-17 Thread GitBox
JiajunBernoulli commented on code in PR #2968: URL: https://github.com/apache/calcite/pull/2968#discussion_r1051511326 ## core/src/main/java/org/apache/calcite/sql2rel/SqlToRelConverter.java: ## @@ -2442,18 +2472,27 @@ private void convertUnnest(Blackboard bb, SqlCall call, @Nu

[GitHub] [calcite] snuyanzin opened a new pull request, #3007: [CALCITE-5440] Bump gradle from 7.4.2 to 7.6

2022-12-17 Thread GitBox
snuyanzin opened a new pull request, #3007: URL: https://github.com/apache/calcite/pull/3007 `sourceCompatibility` and `targetCompatibility` removed in latest gradle. Currently there is one warning in output for `./gradlew help --warning-mode=all` ``` > Task :buildSrc:autostyleKotli

[GitHub] [calcite] mkou commented on pull request #2973: [CALCITE-5180] Implement some of the overloads for BigQuery's DATE and TIMESTAMP

2022-12-17 Thread GitBox
mkou commented on PR #2973: URL: https://github.com/apache/calcite/pull/2973#issuecomment-1356124354 Hey Julian, Yes we agree this is what I meant > > Le 16 déc. 2022 à 18:52, Julian Hyde ***@***.***> a écrit : > >  > @mkou, In SqlLibraryOperators, the TIMESTAMP