[Refactor] Remove IVM FULL refresh mode by Youngwb · Pull Request #72288 · StarRocks/starrocks

Youngwb · 2026-04-28T11:32:30Z

Why I'm doing:

What I'm doing:

user-visible refresh modes are limited to INCREMENTAL and PCT; FULL is removed. FULL has not been released, so no compatibility shim is needed.

Drop FULL from MaterializedView.RefreshMode enum and isFull() helper
Drop the FULL branch from MVRefreshProcessorFactory.newProcessor
Reword the IVM schema-change error to "drop and re-create the MV" since the legacy "trigger a full refresh" path no longer exists
Update affected unit tests and remove the iceberg_full SQL test
Drop the FULL bullet from CREATE MATERIALIZED VIEW docs (en/zh/ja)

Fixes #issue

What type of PR is this:

Does this PR entail a change in behavior?

Yes, this PR will result in a change in behavior.
No, this PR will not result in a change in behavior.

If yes, please specify the type of change:

Interface/UI changes: syntax, type conversion, expression evaluation, display information
Parameter changes: default values, similar parameters but with different default values
Policy changes: use new policy to replace old one, functionality automatically enabled
Feature removed
Miscellaneous: upgrade & downgrade compatibility, etc.

Checklist:

I have added test cases for my bug fix or my new feature
This pr needs user documentation (for new or modified features or behaviors)
- I have added documentation for my new feature or new function
- This pr needs auto generate documentation
This is a backport pr

Bugfix cherry-pick branch check:

Per the IMV V1 PRD, user-visible refresh modes are limited to INCREMENTAL and PCT; FULL is removed. FULL has not been released, so no compatibility shim is needed. - Drop FULL from MaterializedView.RefreshMode enum and isFull() helper - Drop the FULL branch from MVRefreshProcessorFactory.newProcessor - Reword the IVM schema-change error to "drop and re-create the MV" since the legacy "trigger a full refresh" path no longer exists - Update affected unit tests and remove the iceberg_full SQL test - Drop the FULL bullet from CREATE MATERIALIZED VIEW docs (en/zh/ja) Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com> Signed-off-by: Youngwb <yangwenbo_mailbox@163.com>

CelerData-Reviewer · 2026-04-28T11:33:22Z

@codex review

chatgpt-codex-connector

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: d8ed138988

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

Open a pull request for review
Mark a draft as ready
Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

github-actions · 2026-04-28T11:39:02Z

🌎 Translation Required?

✅ All translation files are up to date.
Great job! No translation actions are required for this PR.

🕒 Last updated: Tue, 28 Apr 2026 11:39:01 GMT

github-actions · 2026-04-29T02:44:02Z

[Java-Extensions Incremental Coverage Report]

✅ pass : 0 / 0 (0%)

github-actions · 2026-04-29T02:45:57Z

[FE Incremental Coverage Report]

✅ pass : 0 / 0 (0%)

github-actions · 2026-04-29T02:46:43Z

[BE Incremental Coverage Report]

✅ pass : 0 / 0 (0%)

Youngwb · 2026-04-29T05:55:21Z

@mergify backport branch-4.1

mergify · 2026-04-29T05:55:36Z

backport branch-4.1

✅ Backports have been created

Details

#72321 [Refactor] Remove IVM FULL refresh mode (backport #72288) has been created for branch branch-4.1 but encountered conflicts

Cherry-pick of 0056da9 has failed:

On branch mergify/bp/branch-4.1/pr-72288
Your branch is up to date with 'origin/branch-4.1'.

You are currently cherry-picking commit 0056da9f39.
  (fix conflicts and run "git cherry-pick --continue")
  (use "git cherry-pick --skip" to skip this patch)
  (use "git cherry-pick --abort" to cancel the cherry-pick operation)

Changes to be committed:
	modified:   docs/en/sql-reference/sql-statements/materialized_view/CREATE_MATERIALIZED_VIEW.md
	modified:   docs/ja/sql-reference/sql-statements/materialized_view/CREATE_MATERIALIZED_VIEW.md
	modified:   docs/zh/sql-reference/sql-statements/materialized_view/CREATE_MATERIALIZED_VIEW.md
	modified:   fe/fe-core/src/main/java/com/starrocks/catalog/MaterializedView.java
	modified:   fe/fe-core/src/main/java/com/starrocks/scheduler/mv/BaseMVRefreshProcessor.java
	modified:   fe/fe-core/src/test/java/com/starrocks/alter/AlterMVJobExecutorEditLogTest.java
	modified:   fe/fe-core/src/test/java/com/starrocks/alter/AlterMaterializedViewTest.java
	modified:   fe/fe-core/src/test/java/com/starrocks/scheduler/mv/ivm/IVMBasedMvRefreshProcessorIcebergTest.java
	deleted:    test/sql/test_materialized_view_ivm/R/test_ivm_with_iceberg_full
	deleted:    test/sql/test_materialized_view_ivm/T/test_ivm_with_iceberg_full

Unmerged paths:
  (use "git add <file>..." to mark resolution)
	both modified:   fe/fe-core/src/main/java/com/starrocks/scheduler/mv/MVRefreshProcessorFactory.java

To fix up this pull request, you can check it out locally. See documentation: https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/checking-out-pull-requests-locally

Youngwb · 2026-04-29T08:04:43Z

@mergify backport branch-4.1

mergify · 2026-04-29T08:04:48Z

backport branch-4.1

✅ Backports have been created

Details

#72321 [Refactor] Remove IVM FULL refresh mode (backport #72288) has been created for branch branch-4.1 but encountered conflicts

Cherry-pick of 0056da9 has failed:

On branch mergify/bp/branch-4.1/pr-72288
Your branch is up to date with 'origin/branch-4.1'.

You are currently cherry-picking commit 0056da9f39.
  (fix conflicts and run "git cherry-pick --continue")
  (use "git cherry-pick --skip" to skip this patch)
  (use "git cherry-pick --abort" to cancel the cherry-pick operation)

Changes to be committed:
	modified:   docs/en/sql-reference/sql-statements/materialized_view/CREATE_MATERIALIZED_VIEW.md
	modified:   docs/ja/sql-reference/sql-statements/materialized_view/CREATE_MATERIALIZED_VIEW.md
	modified:   docs/zh/sql-reference/sql-statements/materialized_view/CREATE_MATERIALIZED_VIEW.md
	modified:   fe/fe-core/src/main/java/com/starrocks/catalog/MaterializedView.java
	modified:   fe/fe-core/src/main/java/com/starrocks/scheduler/mv/BaseMVRefreshProcessor.java
	modified:   fe/fe-core/src/test/java/com/starrocks/alter/AlterMVJobExecutorEditLogTest.java
	modified:   fe/fe-core/src/test/java/com/starrocks/alter/AlterMaterializedViewTest.java
	modified:   fe/fe-core/src/test/java/com/starrocks/scheduler/mv/ivm/IVMBasedMvRefreshProcessorIcebergTest.java
	deleted:    test/sql/test_materialized_view_ivm/R/test_ivm_with_iceberg_full
	deleted:    test/sql/test_materialized_view_ivm/T/test_ivm_with_iceberg_full

Unmerged paths:
  (use "git add <file>..." to mark resolution)
	both modified:   fe/fe-core/src/main/java/com/starrocks/scheduler/mv/MVRefreshProcessorFactory.java

To fix up this pull request, you can check it out locally. See documentation: https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/checking-out-pull-requests-locally

Signed-off-by: Youngwb <yangwenbo_mailbox@163.com> Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com> (cherry picked from commit 0056da9)

Signed-off-by: Youngwb <yangwenbo_mailbox@163.com> Co-authored-by: Youngwb <yangwenbo_mailbox@163.com> Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

Signed-off-by: Youngwb <yangwenbo_mailbox@163.com> Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

Youngwb requested a review from a team as a code owner April 28, 2026 11:32

github-actions Bot added the documentation Documentation changes label Apr 28, 2026

mergify Bot assigned Youngwb Apr 28, 2026

github-actions Bot requested a review from stephen-shelby April 28, 2026 11:34

chatgpt-codex-connector Bot reviewed Apr 28, 2026

View reviewed changes

Comment thread fe/fe-core/src/main/java/com/starrocks/catalog/MaterializedView.java

Astralidea approved these changes Apr 29, 2026

View reviewed changes

HangyuanLiu approved these changes Apr 29, 2026

View reviewed changes

HangyuanLiu merged commit 0056da9 into StarRocks:main Apr 29, 2026
82 of 83 checks passed

Youngwb deleted the imv-v1/remove-full-refresh-mode branch April 29, 2026 05:54

mergify Bot mentioned this pull request Apr 29, 2026

[Refactor] Remove IVM FULL refresh mode (backport #72288) #72321

Merged

24 tasks

Youngwb added a commit that referenced this pull request Apr 29, 2026

[Refactor] Remove IVM FULL refresh mode (#72288)

d109222

Signed-off-by: Youngwb <yangwenbo_mailbox@163.com> Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com> (cherry picked from commit 0056da9)

github-actions Bot added the 4.1-merged label Apr 29, 2026

Hyper-FF pushed a commit to Hyper-FF/starrocks that referenced this pull request Apr 30, 2026

[Refactor] Remove IVM FULL refresh mode (StarRocks#72288)

b7dc4eb

Signed-off-by: Youngwb <yangwenbo_mailbox@163.com> Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

robd003 pushed a commit to robd003/starrocks that referenced this pull request May 6, 2026

[Refactor] Remove IVM FULL refresh mode (StarRocks#72288)

0752f0d

Signed-off-by: Youngwb <yangwenbo_mailbox@163.com> Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

[Refactor] Remove IVM FULL refresh mode#72288

[Refactor] Remove IVM FULL refresh mode#72288
HangyuanLiu merged 1 commit into
StarRocks:mainfrom
Youngwb:imv-v1/remove-full-refresh-mode

Youngwb commented Apr 28, 2026 •

edited by github-actions Bot

Loading

Uh oh!

CelerData-Reviewer commented Apr 28, 2026

Uh oh!

chatgpt-codex-connector Bot left a comment

Uh oh!

Uh oh!

github-actions Bot commented Apr 28, 2026

Uh oh!

github-actions Bot commented Apr 29, 2026

Uh oh!

github-actions Bot commented Apr 29, 2026

Uh oh!

github-actions Bot commented Apr 29, 2026

Uh oh!

Uh oh!

Youngwb commented Apr 29, 2026

Uh oh!

mergify Bot commented Apr 29, 2026 •

edited

Loading

Uh oh!

Youngwb commented Apr 29, 2026

Uh oh!

mergify Bot commented Apr 29, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

Uh oh!

Conversation

Youngwb commented Apr 28, 2026 • edited by github-actions Bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Why I'm doing:

What I'm doing:

What type of PR is this:

Checklist:

Bugfix cherry-pick branch check:

Uh oh!

CelerData-Reviewer commented Apr 28, 2026

Uh oh!

chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

💡 Codex Review

Uh oh!

Uh oh!

github-actions Bot commented Apr 28, 2026

🌎 Translation Required?

Uh oh!

github-actions Bot commented Apr 29, 2026

[Java-Extensions Incremental Coverage Report]

Uh oh!

github-actions Bot commented Apr 29, 2026

[FE Incremental Coverage Report]

Uh oh!

github-actions Bot commented Apr 29, 2026

[BE Incremental Coverage Report]

Uh oh!

Uh oh!

Youngwb commented Apr 29, 2026

Uh oh!

mergify Bot commented Apr 29, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

✅ Backports have been created

Uh oh!

Youngwb commented Apr 29, 2026

Uh oh!

mergify Bot commented Apr 29, 2026

✅ Backports have been created

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

Youngwb commented Apr 28, 2026 •

edited by github-actions Bot

Loading

mergify Bot commented Apr 29, 2026 •

edited

Loading