There's quite a lot of ways we can clean up the planner code still. I might not be able to address all of them immediately so tracking them here: - [ ] Remove PinotLogicalAggregate and the corresponding PinotLogicalAggregateRule. We can do the group-trim in the Physical Planning directly. #15439 - [ ] Figure out a better name than Physical* nodes. Currently, I create a partial Physical plan node and incrementally enrich it, which opens up a lot of avenues for bugs to arise and weaker semantics. see #15439
There's quite a lot of ways we can clean up the planner code still. I might not be able to address all of them immediately so tracking them here: