kgyrtkirk opened a new pull request, #17471: URL: https://github.com/apache/druid/pull/17471
* `TestDataBuilder` defines `d1`/`d2` [columns](https://github.com/apache/druid/blob/390c2d68c858ab29388dd56fbbc81847ed204e31/sql/src/test/java/org/apache/druid/sql/calcite/util/TestDataBuilder.java#L393) * `DruidQuery` could be affected by the presence of those columns - as they may interfere with the [prefix selection](https://github.com/apache/druid/blob/390c2d68c858ab29388dd56fbbc81847ed204e31/sql/src/main/java/org/apache/druid/sql/calcite/rel/DruidQuery.java#L468) * `DruidCorrelateUnnestRule` removes a project fully while it [operates](https://github.com/apache/druid/blob/390c2d68c858ab29388dd56fbbc81847ed204e31/sql/src/main/java/org/apache/druid/sql/calcite/rule/DruidCorrelateUnnestRule.java#L155) - which may cause all underlying columns to appear Introduction of `DruidRelFieldTrimmer` which could remove unused columns under `Correlate` cause lots of `_` differences between the decoupled and the old planner as `DruidCorrelateUnnestRule` essentially undoes the trimming -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
