manuzhang opened a new pull request, #905: URL: https://github.com/apache/iceberg-cpp/pull/905
## What changed - add a generic, pull-based `Iterator<T>` whose reads can fail - add `ToVector()` support for move-only and copy-only value types - preserve iterator errors when collecting remaining values - install the public header and add focused unit coverage for collection behavior ## Why This provides a reusable lazy iteration primitive without coupling it to table scan planning. It is extracted from #873 so the iterator contract and the lazy scan implementation can be reviewed independently. PR #873 will consume this utility for lazy manifest and file scan task planning. ## Testing - compiled the public header standalone with C++23 - compiled `ToVector()` instantiations for copy-only and move-only values - added unit tests for copy-only values, move-only values, and error propagation -- 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]
