github-actions[bot] commented on code in PR #68336: URL: https://github.com/apache/doris/pull/68336#discussion_r4061344845
########## fe/fe-core/src/main/java/org/apache/doris/mtmv/ivm/AGENTS.md: ########## @@ -155,7 +155,7 @@ __DORIS_IVM_ROW_ID_COL__ | k1 | cnt | sum_v1 | __DORIS_IVM_DML_FACTOR_COL__ | __ ### Semantics -- **Read-only.** No insert transaction is built. Stream offsets, refresh version, and MV metadata +- **Read-only.** No insert transaction is built. Stream offsets, sequence prefix, and MV metadata Review Comment: Could you finish the terminology sweep? `regression-test/plugins/plugin_planner.groovy:138` and `regression-test/suites/mtmv_p0/ivm/test_ivm_refresh_dry_run.groovy:19` still say the sequence encodes a "refresh version", and the renamed transaction test still uses `*_refresh_version_*` fixture names. Since this refactor is specifically meant to remove that ambiguity, leaving these references keeps teaching the old concept. -- 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]
