zhztheplayer commented on PR #9026: URL: https://github.com/apache/incubator-gluten/pull/9026#issuecomment-2737017007
> What do you think about only insert PartialProject when parent operator requires row format, in another word, not apply partial project when parent operator is ColumnarToRow? @zhztheplayer SGTM if we can add the code directly into PartialProjectRule. A rule like `RemoveColumnarPartialProject` adds more complexity to query planner. Any reason not to use the same rule? -- 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]
