From fb7c42a4e352d9de761b22fb88c2a6c50715cbdc Mon Sep 17 00:00:00 2001
From: tk047 <62949420+tk047@users.noreply.github.com>
Date: Mon, 29 Aug 2022 12:07:31 +0800
Subject: [PATCH] [fix](fe) Fixed alterOp from HashSet to EnumSet (#12094)

Change the HashSet to EnumSet of the AlterOp's currentOps for better performance
---
 .../main/java/org/apache/doris/alter/AlterOperations.java  | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/fe/fe-core/src/main/java/org/apache/doris/alter/AlterOperations.java b/fe/fe-core/src/main/java/org/apache/doris/alter/AlterOperations.java
index 7d43ef59d2..599a7f26d3 100644
--- a/fe/fe-core/src/main/java/org/apache/doris/alter/AlterOperations.java
+++ b/fe/fe-core/src/main/java/org/apache/doris/alter/AlterOperations.java
@@ -23,22 +23,21 @@ import org.apache.doris.common.DdlException;
 import org.apache.doris.common.util.PropertyAnalyzer;
 
 import com.google.common.base.Joiner;
-import com.google.common.collect.Sets;
 
+import java.util.EnumSet;
 import java.util.List;
-import java.util.Set;
 
 /*
  * AlterOperations contains a set alter operations generated from a AlterStmt's alter clause.
  * This class is mainly used to integrate these operation types and check whether they have conflicts.
  */
 public class AlterOperations {
-    private Set<AlterOpType> currentOps = Sets.newHashSet();
+    private EnumSet<AlterOpType> currentOps = EnumSet.noneOf(AlterOpType.class);
 
     public AlterOperations() {
     }
 
-    public Set<AlterOpType> getCurrentOps() {
+    public EnumSet<AlterOpType> getCurrentOps() {
         return currentOps;
     }