jeremiahrhall opened a new pull request #2204: URL: https://github.com/apache/calcite/pull/2204
Hi folks. I was excited to see pivot support merged and decided to try it out. I found that it works great for the CSV adapter and Reflective schema, but did not get the results I expected when using a JDBC adapter. Digging into I found that the SQL issued to the database was lacking the `FILTER (WHERE ...)` expressions. Digging further, I found that `FILTER (WHERE ...)` relational expressions appear to be ignored by `SqlImplementor.java` in all cases that I tried. This change wraps the aggregate call with a filter when present. Also filing an issue in Jira momentarily. ---------------------------------------------------------------- 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 comment. For queries about this service, please contact Infrastructure at: [email protected]
