[GitHub] [calcite] James-Jeyun-Kim commented on pull request #2046: [CALCITE-4092] NPE using WITH clause without a corresponding SELECT F…

2020-06-26 Thread GitBox
James-Jeyun-Kim commented on pull request #2046: URL: https://github.com/apache/calcite/pull/2046#issuecomment-650405565 I deleted the overridden function "resolve" in WithScope.java because that was adding the view from the WITH clause into the scope of the main statement. This made the q

[GitHub] [calcite] James-Jeyun-Kim commented on pull request #2046: [CALCITE-4092] NPE using WITH clause without a corresponding SELECT F…

2020-06-26 Thread GitBox
James-Jeyun-Kim commented on pull request #2046: URL: https://github.com/apache/calcite/pull/2046#issuecomment-650403210 This PR catches an invalid WITH clause during the query validation step so that it doesn't lead to a NPE later on in the sqlToRelConverter stage. --