Github user smola commented on the pull request:

    https://github.com/apache/spark/pull/6463#issuecomment-106525540
  
    This is still a work in progress.
    
    @rxin I would like some feedback about how to continue with the 
implementation. In ExpressionBuilders I added some helper methods that can 
convert a Expression into a function name and a function builder (i.e. 
`(Seq[Expression]) => Expression`). I added two alternative implementations, 
one with macros and one with reflection. How does it look?
    
    Any other feedback will be really appreciated. Thanks.


---
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