ajantha-bhat commented on PR #6661: URL: https://github.com/apache/iceberg/pull/6661#issuecomment-1450306142
@szehon-ho: a) I have addressed the duplicate delete file problem using `HashSet` instead of `PositionDeletesScan` because we don't have a similar logic for equality deletes and code looks odd. Maybe later we can replace the logic as it is internal logic. b) I have derived the refactoring work of this PR into a separate one #6975 to reduce the review effort on this PR (we need to merge this first) -- 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
