diff --git a/semantic/query/src/main/java/com/tencent/supersonic/semantic/query/parser/convert/CalculateAggConverter.java b/semantic/query/src/main/java/com/tencent/supersonic/semantic/query/parser/convert/CalculateAggConverter.java
index 1c6552578..02e47b149 100644
--- a/semantic/query/src/main/java/com/tencent/supersonic/semantic/query/parser/convert/CalculateAggConverter.java
+++ b/semantic/query/src/main/java/com/tencent/supersonic/semantic/query/parser/convert/CalculateAggConverter.java
@@ -23,7 +23,6 @@ import java.util.List;
 import java.util.Objects;
 import java.util.stream.Collectors;
 import lombok.extern.slf4j.Slf4j;
-import org.apache.commons.lang3.StringUtils;
 import org.springframework.beans.factory.annotation.Value;
 import org.springframework.context.annotation.Lazy;
 import org.springframework.stereotype.Component;
@@ -123,7 +122,7 @@ public class CalculateAggConverter implements SemanticConverter {
 
     @Override
     public void converter(Catalog catalog, QueryStructReq queryStructCmd, ParseSqlReq sqlCommend,
-                          MetricReq metricCommand) throws Exception {
+            MetricReq metricCommand) throws Exception {
         DatabaseResp databaseResp = catalog.getDatabaseByModelId(queryStructCmd.getModelId());
         ParseSqlReq parseSqlReq = generateSqlCommend(queryStructCmd,
                 EngineTypeEnum.valueOf(databaseResp.getType().toUpperCase()), databaseResp.getVersion());
@@ -150,7 +149,7 @@ public class CalculateAggConverter implements SemanticConverter {
     }
 
     public ParseSqlReq generateRatioSqlCommand(QueryStructReq queryStructCmd, EngineTypeEnum engineTypeEnum,
-                                               String version)
+            String version)
             throws Exception {
         check(queryStructCmd);
         ParseSqlReq sqlCommand = new ParseSqlReq();
@@ -384,7 +383,7 @@ public class CalculateAggConverter implements SemanticConverter {
     }
 
 
-    private static String getAllJoinSelect(QueryStructReq queryStructCmd, String alias) {
+    private String getAllJoinSelect(QueryStructReq queryStructCmd, String alias) {
         String aggStr = queryStructCmd.getAggregators().stream()
                 .map(f -> getSelectField(f, alias) + " as " + getSelectField(f, "")
                         + "_roll")
@@ -418,23 +417,18 @@ public class CalculateAggConverter implements SemanticConverter {
     }
 
 
-    private static String getAllSelect(QueryStructReq queryStructCmd, String alias) {
+    private String getAllSelect(QueryStructReq queryStructCmd, String alias) {
         String aggStr = queryStructCmd.getAggregators().stream().map(f -> getSelectField(f, alias))
                 .collect(Collectors.joining(","));
         return CollectionUtils.isEmpty(queryStructCmd.getGroups()) ? aggStr
                 : alias + String.join("," + alias, queryStructCmd.getGroups()) + "," + aggStr;
     }
 
-    private static String getSelectField(final Aggregator agg, String alias) {
+    private String getSelectField(final Aggregator agg, String alias) {
         if (agg.getFunc().equals(AggOperatorEnum.RATIO_OVER) || agg.getFunc().equals(AggOperatorEnum.RATIO_ROLL)) {
             return alias + agg.getColumn();
         }
-        if (CollectionUtils.isEmpty(agg.getArgs())) {
-            return agg.getFunc() + "( " + alias + agg.getColumn() + " ) AS " + agg.getColumn() + " ";
-        }
-        return agg.getFunc() + "( " + agg.getArgs().stream().map(arg ->
-                arg.equals(agg.getColumn()) ? arg : (StringUtils.isNumeric(arg) ? arg : ("'" + arg + "'"))
-        ).collect(Collectors.joining(",")) + " ) AS " + agg.getColumn() + " ";
+        return sqlGenerateUtils.getSelectField(agg);
     }
 
     private String getGroupBy(QueryStructReq queryStructCmd) {
diff --git a/semantic/query/src/main/java/com/tencent/supersonic/semantic/query/utils/SqlGenerateUtils.java b/semantic/query/src/main/java/com/tencent/supersonic/semantic/query/utils/SqlGenerateUtils.java
index 9ebd14765..7fdab66e8 100644
--- a/semantic/query/src/main/java/com/tencent/supersonic/semantic/query/utils/SqlGenerateUtils.java
+++ b/semantic/query/src/main/java/com/tencent/supersonic/semantic/query/utils/SqlGenerateUtils.java
@@ -1,14 +1,15 @@
 package com.tencent.supersonic.semantic.query.utils;
 
+import static com.tencent.supersonic.common.pojo.Constants.JOIN_UNDERLINE;
+
+import com.tencent.supersonic.common.pojo.Aggregator;
 import com.tencent.supersonic.semantic.api.model.enums.TimeDimensionEnum;
 import com.tencent.supersonic.semantic.api.query.request.QueryStructReq;
-import com.tencent.supersonic.common.pojo.Aggregator;
 import java.util.stream.Collectors;
 import lombok.extern.slf4j.Slf4j;
 import org.apache.commons.lang3.StringUtils;
 import org.springframework.stereotype.Component;
 import org.springframework.util.CollectionUtils;
-import static com.tencent.supersonic.common.pojo.Constants.JOIN_UNDERLINE;
 
 @Component
 @Slf4j
@@ -54,9 +55,9 @@ public class SqlGenerateUtils {
 
     public String getSelectField(final Aggregator agg) {
         if (CollectionUtils.isEmpty(agg.getArgs())) {
-            return agg.getFunc() + " ( " + agg.getColumn() + " ) AS " + agg.getColumn() + " ";
+            return agg.getFunc() + "( " + agg.getColumn() + " ) AS " + agg.getColumn() + " ";
         }
-        return agg.getFunc() + " ( " + agg.getArgs().stream().map(arg ->
+        return agg.getFunc() + "( " + agg.getArgs().stream().map(arg ->
                 arg.equals(agg.getColumn()) ? arg : (StringUtils.isNumeric(arg) ? arg : ("'" + arg + "'"))
         ).collect(Collectors.joining(",")) + " ) AS " + agg.getColumn() + " ";
     }