kbendick commented on pull request #3213: URL: https://github.com/apache/iceberg/pull/3213#issuecomment-946019086
> > > @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🙂](https://github.com/apache/iceberg/pull/3213#discussion_r724905119%F0%9F%99%82) Hi @myandpr! Sorry I was also out of office for a bit so catching up on things still somewhat. Will take a look ASAP! -- 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]
