jackye1995 commented on pull request #1786:
URL: https://github.com/apache/iceberg/pull/1786#issuecomment-730679823


   @danielcweeks I ran some tests and here is the result:
   1M average with reflection: 14.720822923822924 us
   1M average with function: 2.183976667976668 us
   1M average with base: 2.0982427002427 us
   
   So the current function-based approach should provide a balance between 
clean interface and performance. Please let me know what you think.
   
   P.S. for reflection approach, I use `DynMethods.UnboundMethod` to find the 
methods, and cache the method after creation for repeated invocation.


----------------------------------------------------------------
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:
[email protected]



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

Reply via email to