RussellSpitzer commented on PR #9822: URL: https://github.com/apache/iceberg/pull/9822#issuecomment-2287055070
I agree with @Fokko here, I'm a little worried about adding manifests directly since those are strictly tied to format versions and adding an incorrectly written manifest would break the whole table. If we did have these apis I also think they probably would want to be part of a separate top level API but as of now I think i'm -1 on the idea. If the issue is heap memory pressure is there any way we could just have implementations more eagerly write out manifests? Seems like something that could be fixed at the implementation level rather than introducing a new API. -- 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-unsubscr...@iceberg.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org For additional commands, e-mail: issues-h...@iceberg.apache.org
