xloya opened a new pull request #3842: URL: https://github.com/apache/iceberg/pull/3842
We use the starting-sequence-number configuration when rewriting, then the data written in this rewrite will set the `status` to `ADDED` in the manifest entry, and set the `sequence_number` to the sequence number of the snapshot read before the rewrite. If this rewrite also triggers the `manifest merge` at the end, the sequence number of the new added data file's manifest entry will be lost. For example: We rewrite the snapshot with sequence number equals to `1` and enable `use-starting-sequence-number`, which triggers the rewriting of some data files and the merging of manifests. It is expected that the sequence number of the new manifest entries of the new rewritten data files should be `1`. When actually rewriting, because the manifest merge is triggered, the current logic sets the sequence number to `null`, which does not meet the actual expectations. cc @rdblue @jackye1995 @RussellSpitzer -- 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]
