[GitHub] [calcite-avatica] julianhyde commented on a diff in pull request #205: [CALCITE-5487] Proper semantics for TIMESTAMP WITH LOCAL TIME ZONE

2023-01-19 Thread GitBox
julianhyde commented on code in PR #205: URL: https://github.com/apache/calcite-avatica/pull/205#discussion_r1081992148 ## core/src/main/java/org/apache/calcite/avatica/util/AbstractCursor.java: ## @@ -238,12 +244,18 @@ protected Accessor createAccessor(ColumnMetaData

[GitHub] [calcite-avatica] julianhyde commented on a diff in pull request #205: [CALCITE-5487] Proper semantics for TIMESTAMP WITH LOCAL TIME ZONE

2023-01-19 Thread GitBox
julianhyde commented on code in PR #205: URL: https://github.com/apache/calcite-avatica/pull/205#discussion_r1081983370 ## core/src/main/java/org/apache/calcite/avatica/AvaticaResultSet.java: ## @@ -77,7 +78,9 @@ public AvaticaResultSet(AvaticaStatement statement,

[GitHub] [calcite-avatica] julianhyde commented on a diff in pull request #205: [CALCITE-5487] Proper semantics for TIMESTAMP WITH LOCAL TIME ZONE

2023-01-19 Thread GitBox
julianhyde commented on code in PR #205: URL: https://github.com/apache/calcite-avatica/pull/205#discussion_r1081816343 ## core/src/main/java/org/apache/calcite/avatica/AvaticaResultSet.java: ## @@ -77,7 +78,9 @@ public AvaticaResultSet(AvaticaStatement statement,