chucheng92 commented on PR #3394:
URL: https://github.com/apache/calcite/pull/3394#issuecomment-1690948457

   Hi, @julianhyde function cache is a really useful optimization, and note 
that `RexImpTable`
   many of the previous string methods have been unified into 
`BuiltInMethod.method`, which seems to be a very good refactor, neat and 
consistent.
   
   only left a few comments.


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

Reply via email to