iskakaushik opened a new pull request, #252: URL: https://github.com/apache/iceberg-go/pull/252
This refactor breaks down the manifest scanning logic into more focused components in preparation for adding incremental scanners to allow reading changelog and diffs between two snapshots. Key changes include: - Add manifestEntries type to safely collect data and delete entries concurrently - Split manifest handling into separate fetchPartitionSpecFilteredManifests and collectManifestEntries functions for better separation of concerns - Replace manual goroutine management with errgroup for more robust concurrency - Add documentation comments explaining the manifest scanning process This is a step toward adding a ManifestGroup abstraction similar to the Java implementation that can be shared among different scanner types. -- 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
