seawinde opened a new pull request, #67281:
URL: https://github.com/apache/doris/pull/67281
### What problem does this PR solve?
Issue Number: N/A
Related PR: N/A
Problem Summary:
`REFRESH MATERIALIZED VIEW ... COMPLETE` creates an internal DML plan. When
`enable_dml_materialized_view_rewrite` is disabled, that plan must not
register
the DML materialized view rewrite hook or collect other MTMVs as rewrite
candidates. A refresh-success-only test cannot detect the unwanted candidate
collection because refresh may still complete successfully.
Root cause: regression protection for the planner hook and candidate
collection
path introduced by PR #48374 was missing.
| File | Change Description |
|------|--------------------|
| `AddInitMaterializationHookTest.java` | Covers the query/DML rewrite
switch matrix and verifies DML hook registration plus `getCandidateMTMVs()`
invocation count. |
| `test_dml_materialized_view_rewrite_global.groovy` | Covers the GLOBAL
variable, inheritance by a new connection, asynchronous complete refresh,
result correctness, and variable restoration. |
This is test-only coverage. No production behavior is changed.
### Release note
None
### Check List (For Author)
- Test
- [ ] Regression test
- [ ] Unit Test
- [ ] Manual test (add detailed scripts or steps below)
- [x] No need to test or manual test. Explain why:
- [ ] This is a refactor/code format and no logic has been changed.
- [ ] Previous test can cover this change.
- [x] No code files have been changed.
- [x] Other reason
The FE unit test source passed a local `javac` API check against existing
build
artifacts. The targeted Maven run was blocked before compilation because
local
Doris module artifacts referenced through `${revision}` were unavailable. The
regression case was not run because no restarted Doris test cluster was
available.
- Behavior changed:
- [x] No.
- [ ] Yes.
- Does this need documentation?
- [x] No.
- [ ] Yes.
### Check List (For Reviewer who merge this PR)
- [ ] Confirm the release note
- [ ] Confirm test cases
- [ ] Confirm document
- [ ] Add branch pick label
--
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]
