You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@iceberg.apache.org by GitBox <gi...@apache.org> on 2022/07/07 05:37:49 UTC

[GitHub] [iceberg] rizaon commented on pull request #4518: core: Provide mechanism to cache manifest file content

rizaon commented on PR #4518:
URL: https://github.com/apache/iceberg/pull/4518#issuecomment-1177100084

   Hi @danielcweeks , thank you for the update. I will check PR #5207 and study how we can utilize it.
   
   So with that PR, am I correct that we should implement the caching in ManifestFiles.java instead of creating a new CachingFileIO.java as this PR did?


-- 
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: issues-unsubscribe@iceberg.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org