From 530bacb26e79d640be86099ae096659288779caa Mon Sep 17 00:00:00 2001 From: Jibing Li Date: Wed, 29 Nov 2023 16:33:48 +0800 Subject: [PATCH] Fix show auto analyze missing jobs bug. --- .../apache/doris/statistics/AnalysisManager.java | 2 +- .../doris/statistics/AnalysisManagerTest.java | 13 +++++++++++++ 2 files changed, 14 insertions(+), 1 deletion(-) diff --git a/fe/fe-core/src/main/java/org/apache/doris/statistics/AnalysisManager.java b/fe/fe-core/src/main/java/org/apache/doris/statistics/AnalysisManager.java index 61c72926f2d84a..cd63379235467a 100644 --- a/fe/fe-core/src/main/java/org/apache/doris/statistics/AnalysisManager.java +++ b/fe/fe-core/src/main/java/org/apache/doris/statistics/AnalysisManager.java @@ -1058,7 +1058,7 @@ protected SimpleQueue createSimpleQueue(Collection c a -> { // DO NOTHING return null; - }, null); + }, collection); } // Remove col stats status from TableStats if failed load some col stats after analyze corresponding column so that diff --git a/fe/fe-core/src/test/java/org/apache/doris/statistics/AnalysisManagerTest.java b/fe/fe-core/src/test/java/org/apache/doris/statistics/AnalysisManagerTest.java index afce13df7a3ad6..72754943404de4 100644 --- a/fe/fe-core/src/test/java/org/apache/doris/statistics/AnalysisManagerTest.java +++ b/fe/fe-core/src/test/java/org/apache/doris/statistics/AnalysisManagerTest.java @@ -29,6 +29,7 @@ import org.apache.doris.statistics.AnalysisInfo.AnalysisType; import org.apache.doris.statistics.AnalysisInfo.JobType; import org.apache.doris.statistics.AnalysisInfo.ScheduleType; +import org.apache.doris.statistics.util.SimpleQueue; import org.apache.doris.statistics.util.StatisticsUtil; import com.google.common.annotations.VisibleForTesting; @@ -445,4 +446,16 @@ public void testRecordLimit3() { Assertions.assertEquals(2, analysisManager.autoJobs.size()); } + @Test + public void testCreateSimpleQueue() { + AnalysisManager analysisManager = new AnalysisManager(); + ArrayList jobs = Lists.newArrayList(); + jobs.add(new AnalysisInfoBuilder().setJobId(1).build()); + jobs.add(new AnalysisInfoBuilder().setJobId(2).build()); + SimpleQueue simpleQueue = analysisManager.createSimpleQueue(jobs, analysisManager); + Assertions.assertEquals(2, simpleQueue.size()); + simpleQueue = analysisManager.createSimpleQueue(null, analysisManager); + Assertions.assertEquals(0, simpleQueue.size()); + } + }