Github user chenghao-intel commented on the pull request:

    https://github.com/apache/spark/pull/9480#issuecomment-154599464
  
    >
    I'm still not sure how you are suggesting we execute the version you are 
proposing. I'd guess you do that by adding an additional project that creates a 
new intermediate row and substitute the common subexpression to read from that 
new intermediate row. I think this is bad for performance. One of the big 
benefits of codegen is to remove those "unnecessary" intermediate rows.
    >
    
    I don't think I will do that in the way you described, actually we are 
working on the common xxx elimination right now, probably more information will 
be shared later, will ping you then.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to