parthchandra commented on PR #4891: URL: https://github.com/apache/datafusion-comet/pull/4891#issuecomment-4940747083
> I was thinking about another approach for this. I haven't explored how practical it is to implement though. > > The extended explain plan just shows operators, but what if we include expressions and use a symbol to annotate whether the expression is rust or jvm? > > For example, here `*` could mean Rust and `+` could mean JVM. > > ``` > CometProject: expr1*, expr2* expr3+, expr4+ > CometFilter: expr* > ``` The way to do it, I think, would be to have a new tag for the expression nodes and then the explain plan generator can format it however we want it to be displayed. -- 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]
