gaborkaszab commented on PR #16849: URL: https://github.com/apache/iceberg/pull/16849#issuecomment-4732616891
Currently, there is no mechanism to set `Tracking.firstRowId`. Following the PR for Tracking builder, [it was commented](https://github.com/apache/iceberg/pull/16092/changes#r3244858155) that setting `firstRowId` shouldn't be through the builder but via inheritance. This PR adds a simple setter function that can be used for this. If I understand correctly, it wouldn't suite `inheritFrom(manifestTracking)` because it's not a direct inheritance, but based calculated based on other manifest entries that has null `firstRowId` + the manifest's `firstRowId`. -- 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]
