kuldeep27396 opened a new pull request, #15624: URL: https://github.com/apache/iceberg/pull/15624
## Why this change Resolves #15622 **Problem**: was creating a new wrapper on every invocation via . When file metadata is read and rewritten (e.g., during manifest rewriting or format conversion), each entry needlessly allocates a list that is immediately consumed and discarded. **Solution**: Cache the representation after the first conversion, similar to how other fields like are handled. The cache is transient so it won't be serialized. ## Changes - Added field to cache the converted list - Modified to return the cached list instead of creating a new one on each call - The cached list is created lazily on first access ## Performance Impact This eliminates unnecessary object allocations when: - Reading and rewriting manifest files - Format conversions - Any scenario where is called multiple times on the same instance The change is minimal and focused, following the existing pattern used for other cached fields in the class. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
