NikhilCollooru opened a new issue, #6332: URL: https://github.com/apache/hudi/issues/6332
We observed that creating Configuration copies is consuming a lot of CPU. We made changes in Presto to use a wrapper instead of creating configuration copies. But we were told that the presto change is breaking Hudi. So one suggestion is to avoid creating copies in places like this: https://github.com/apache/hudi/blob/master/hudi-common/src/main/java/org/apache/hudi/common/config/SerializableConfiguration.java#L37 we can instead simply do ``` public SerializableConfiguration(Configuration configuration) { this.configuration = configuration; } ``` The breaking Presto PR: https://github.com/prestodb/presto/pull/18115 https://github.com/prestodb/presto/issues/17736 @pratyakshsharma @7c00 do you the code suggestion above makes sense ? -- 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]
