Github user twalthr commented on the pull request:

    https://github.com/apache/flink/pull/1556#issuecomment-176157407
  
    As far as I know the `Expressions` are for the user-facing API only, right? 
I think we won't need the `RexToExpr` class anymore, as I am currently writing 
a rex to code generator.
    
    You have implemented `Function` with row types by default. In the old Table 
API we also generated these functions in order to not only support row types. 
In the future we could also use Tuple type where possible to improve the 
performance. Do we want lose this possibility? 


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Reply via email to