myandpr commented on pull request #3213: URL: https://github.com/apache/iceberg/pull/3213#issuecomment-938975578
> > @rdblue @kbendick @stevenzwu , could you help to review this PR? :) > > Thanks for working on this @myandpr. I am off this week but am interested in this so will review but might be a bit slow to respond. 🙂 > > I agree with @rdblue's that it makes sense to avoid trying to compact the whole table (or scan the whole table) in an online streaming context, but rather pass around the file names that need to be compacted and committed. Hi @kbendick , I'd like to hear your opinion on my doubts https://github.com/apache/iceberg/pull/3213#discussion_r724905119🙂 -- 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]
