JingsongLi commented on PR #8256: URL: https://github.com/apache/paimon/pull/8256#issuecomment-4737336969
> @JingsongLi They are related only in that both touch sequence metadata, but they solve different problems. #7832 changes merge ordering semantics by using snapshot id as the ordering sequence. This PR does not change ordering semantics; it only optimizes how a write-only primary-key writer initializes the next sequence number, by persisting the global max sequence number in snapshot properties and avoiding scanning existing file metadata after bootstrap. This is super confusing from the config option name. We may need a better name, for example, commit.record-max-sequence, something like this, you can use AI to find a new one. -- 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]
