rubenada commented on PR #2747: URL: https://github.com/apache/calcite/pull/2747#issuecomment-1216902172
Thanks for the feedback @zgramana > This PR might be worth merging to catch any potential regressions. IMO there's no need for that, the PR that fixed [CALCITE-5061](https://issues.apache.org/jira/browse/CALCITE-5061) (and as a side effect also [CALCITE-5051](https://issues.apache.org/jira/browse/CALCITE-5051) ) included a test that is very similar to the one proposed in this PR: https://github.com/apache/calcite/blob/7de37aefaaa72959a0145053319fae2956fbb9f1/core/src/test/java/org/apache/calcite/sql2rel/RelFieldTrimmerTest.java#L520 -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
