yujun777 opened a new pull request, #68336: URL: https://github.com/apache/doris/pull/68336
`IvmInfo.refreshVersion` is the high part of the sequence values an IVM MV's rows are stamped with -- the low part next to it is a delta index. The name reads like an epoch or like the MV's own version, which is what the per-partition `refreshEpoch` being added alongside it is not, and the two sitting in the same code base is a trap for the next reader. This renames it to `sequencePrefix`: the prefix of the `(sequence prefix, delta index, op)` triple that `IvmSequenceCalculator` encodes into the sequence column. Nothing about the value changes -- it still counts committing IVM transactions and still prefixes the sequence column. - field and accessors: `sequencePrefix`, `getSequencePrefix()`, `advanceSequencePrefix()` - `MTMV.getNextRefreshVersion()` -> `MTMV.getNextSequencePrefix()` - `IvmSequenceCalculator` identifiers, including `LARGEINT_SEQUENCE_PREFIX_SHIFT` and the range-check messages - the persisted name changes outright from `"rv"` to `"sp"`: IVM is not released, so there is no image or journal in the wild that writes the old name Trace: https://github.com/apache/doris/issues/65418 -- 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]
