You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pinot.apache.org by GitBox <gi...@apache.org> on 2020/11/05 02:18:01 UTC

[GitHub] [incubator-pinot] fx19880617 commented on issue #5919: Add endpoint for external to get information about UDFs

fx19880617 commented on issue #5919:
URL: https://github.com/apache/incubator-pinot/issues/5919#issuecomment-722082393


   > How do you plan to maintain this endpoint?
   > Not sure if this can help with query push down as the behavior must be identical on presto side and pinot side. IMO it is safer to manually select functions to push down instead of using name matching. It can cause unexpected behavior if the behavior is not exactly the same.
   
   We can use reflection to auto generate this information or we can get it from TransformFunctionFactory. 
   I think this is useful when we have many releases of Pinot and Presto and we are keep adding new functions.
   
   Also cc: @yupeng9 
   


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

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org