[doris] branch master updated: [fix](mtmv) fix bug that should not write edit log when replaying alter mv (#19781)

[morningman]

This is an automated email from the ASF dual-hosted git repository.

morningman pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/doris.git


The following commit(s) were added to refs/heads/master by this push:
     new e45bc160c9 [fix](mtmv) fix bug that should not write edit log when 
replaying alter mv (#19781)
e45bc160c9 is described below

commit e45bc160c9e0565d722f6084ed75a45f577e0794
Author: Mingyu Chen <morning...@163.com>
AuthorDate: Thu May 18 13:34:05 2023 +0800

    [fix](mtmv) fix bug that should not write edit log when replaying alter mv 
(#19781)
---
 fe/fe-core/src/main/java/org/apache/doris/alter/Alter.java         | 2 +-
 fe/fe-core/src/main/java/org/apache/doris/mtmv/MTMVJobManager.java | 4 ++--
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/fe/fe-core/src/main/java/org/apache/doris/alter/Alter.java 
b/fe/fe-core/src/main/java/org/apache/doris/alter/Alter.java
index bd4b86b1fd..04dab7139a 100644
--- a/fe/fe-core/src/main/java/org/apache/doris/alter/Alter.java
+++ b/fe/fe-core/src/main/java/org/apache/doris/alter/Alter.java
@@ -529,7 +529,7 @@ public class Alter {
             olapTable = (MaterializedView) 
db.getTableOrMetaException(tbl.getTbl(), TableType.MATERIALIZED_VIEW);
 
             // 2. drop old job and kill the associated tasks
-            Env.getCurrentEnv().getMTMVJobManager().dropJobByName(tbl.getDb(), 
tbl.getTbl());
+            Env.getCurrentEnv().getMTMVJobManager().dropJobByName(tbl.getDb(), 
tbl.getTbl(), isReplay);
 
             // 3. overwrite the refresh info in the memory of fe.
             olapTable.writeLock();
diff --git a/fe/fe-core/src/main/java/org/apache/doris/mtmv/MTMVJobManager.java 
b/fe/fe-core/src/main/java/org/apache/doris/mtmv/MTMVJobManager.java
index 44df43a8e3..ae3d9007a6 100644
--- a/fe/fe-core/src/main/java/org/apache/doris/mtmv/MTMVJobManager.java
+++ b/fe/fe-core/src/main/java/org/apache/doris/mtmv/MTMVJobManager.java
@@ -367,11 +367,11 @@ public class MTMVJobManager {
         LOG.info("change job:{}", changeJob.getJobId());
     }
 
-    public void dropJobByName(String dbName, String mvName) {
+    public void dropJobByName(String dbName, String mvName, boolean isReplay) {
         for (String jobName : nameToJobMap.keySet()) {
             MTMVJob job = nameToJobMap.get(jobName);
             if (job.getMVName().equals(mvName) && 
job.getDBName().equals(dbName)) {
-                dropJobs(Collections.singletonList(job.getId()), false);
+                dropJobs(Collections.singletonList(job.getId()), isReplay);
                 return;
             }
         }


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org