check table type

Signed-off-by: Murphy <[email protected]>
murphyatwork · Dec 5, 2024 · 29fcc50 · 29fcc50
1 parent 2b797ed
commit 29fcc50
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 7 deletions.
diff --git a/fe/fe-core/src/main/java/com/starrocks/common/Config.java b/fe/fe-core/src/main/java/com/starrocks/common/Config.java
@@ -1973,9 +1973,6 @@ public class Config extends ConfigBase {
     @ConfField(mutable = true)
     public static String statistic_auto_analyze_end_time = "23:59:59";
 
-    @ConfField(mutable = true, comment = "The interval of auto analyze regular columns")
-    public static String statistic_auto_analyze_regular_column_interval_hours = "12";
-
     /**
      * a period of create statistics table automatically by the StatisticsMetaManager
      */
@@ -2019,10 +2016,6 @@ public class Config extends ConfigBase {
             "columns after this period")
     public static long statistic_predicate_columns_ttl_hours = 24;
 
-    @ConfField(mutable = true, comment = "If > 0 it will be the threshold of predicate columns. Otherwise the " +
-            "it will be used to turn off this function")
-    public static long statistic_predicate_columns_threshold = 32;
-
     /**
      * Num of thread to handle statistic collect(analyze command)
      */

diff --git a/fe/fe-core/src/main/java/com/starrocks/sql/analyzer/AnalyzeStmtAnalyzer.java b/fe/fe-core/src/main/java/com/starrocks/sql/analyzer/AnalyzeStmtAnalyzer.java
@@ -159,6 +159,11 @@ public Void visitAnalyzeStatement(AnalyzeStmt statement, ConnectContext session)
 
             // ANALYZE TABLE xxx PREDICATE COLUMNS
             if (statement.isUsePredicateColumns()) {
+                // check if the table type is supported
+                if (!analyzeTable.isNativeTableOrMaterializedView()) {
+                    throw new SemanticException("Only OLAP table can support ANALYZE PREDICATE COLUMNS");
+                }
+
                 List<String> targetColumns = Lists.newArrayList();
 
                 List<ColumnUsage> predicateColumns =