yujun777 opened a new pull request, #67646:
URL: https://github.com/apache/doris/pull/67646
This PR contains two follow-ups from the #62606 review round:
1. **Derive IVM hidden column names from `IVM_HIDDEN_COLUMN_PREFIX`** — IVM
hidden column names are now derived from the shared prefix constant instead of
duplicated literals.
2. **Fail aggregate IVM refresh when the previous refresh txn is not visible
yet** — an aggregate delta joins the MV's old rows with the new delta; when the
previous refresh txn committed but its data is not visible yet (an MV
partition's committed version is ahead of its visible version), that join
misses the old rows and permanently loses the delta. The delta rewriter now
fails a non-empty aggregate delta with `MV_COMMIT_NOT_VISIBLE` and the refresh
falls back to a rebuild that recomputes from the base tables without reading
old MV state.
Adds a nonConcurrent regression that holds a refresh txn in COMMITTED via
the new `DatabaseTransactionMgr.finishTransaction.block_visible` debug point,
then verifies the next strict incremental refresh fails while a COMPLETE
refresh can still report SUCCESS with its txn not yet visible, and that
everything converges once the stuck txns publish.
Trace issue: 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]
