jackye1995 commented on pull request #1786: URL: https://github.com/apache/iceberg/pull/1786#issuecomment-730208438
> This has a lot of overlap with work done in #1767. I'd prefer to hold off on this until work there is done since it switches to using more than just get and put operations and needs to handle encryption for mutlipart operations. Yes, I am thinking if we can solve the issue here and rebase that PR so that PR can be more concise. Let me run some performance check tomorrow to see how much impact it has if we use dynamic method loader to do it versus the current way using function interface. ---------------------------------------------------------------- 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]
