Neuw84 commented on PR #17296: URL: https://github.com/apache/iceberg/pull/17296#issuecomment-5104851101
@amogh-jahagirdar, if you have some time please look at the comments. I discovered a leak (fixed in this PR) but we have some doubts on the previous logic on the Arrow readers for row lineage readers. Let us know!, my opinion is that if the change ( from your comments) is big, lets merge this as is critical ( basically V3 on Structured Streaming is broken), I will open another one for that... if it is small change after you clarification I will implement the changes. -- 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]
