[GitHub] [calcite] libenchao commented on a diff in pull request #2918: [CALCITE-5297] Casting dynamic variable twice throws exception

2022-09-27 Thread GitBox
libenchao commented on code in PR #2918: URL: https://github.com/apache/calcite/pull/2918#discussion_r981219448 ## core/src/main/java/org/apache/calcite/sql/validate/SqlValidatorImpl.java: ## @@ -235,8 +235,11 @@ public class SqlValidatorImpl implements SqlValidatorWithHints {

[GitHub] [calcite] libenchao commented on a diff in pull request #2918: [CALCITE-5297] Casting dynamic variable twice throws exception

2022-09-25 Thread GitBox
libenchao commented on code in PR #2918: URL: https://github.com/apache/calcite/pull/2918#discussion_r979405386 ## core/src/main/java/org/apache/calcite/sql/validate/SqlValidatorImpl.java: ## @@ -235,8 +235,11 @@ public class SqlValidatorImpl implements SqlValidatorWithHints {