refactor(model): default measure agg to none.

- 添加对 dataTypeMap 的 containsKey 检查避免空指针异常
- 修复维度数据类型映射逻辑中的潜在运行时错误
- 确保在模型ID不存在时跳过数据类型映射操作

common/src/main/java/com/tencent/supersonic/common/calcite/SqlDialectFactory.java

@@ -26,12 +26,10 @@ public class SqlDialectFactory {
                     .withLiteralQuoteString("'").withIdentifierQuoteString("\"")
                     .withLiteralEscapedQuoteString("''").withUnquotedCasing(Casing.UNCHANGED)
                     .withQuotedCasing(Casing.UNCHANGED).withCaseSensitive(true);
-    public static final Context PRESTO_CONTEXT =
-            SqlDialect.EMPTY_CONTEXT.withDatabaseProduct(DatabaseProduct.PRESTO)
-                    .withLiteralQuoteString("'")
-                    .withLiteralEscapedQuoteString("''")
-                    .withUnquotedCasing(Casing.UNCHANGED)
-                    .withQuotedCasing(Casing.UNCHANGED).withCaseSensitive(true);
+    public static final Context PRESTO_CONTEXT = SqlDialect.EMPTY_CONTEXT
+            .withDatabaseProduct(DatabaseProduct.PRESTO).withLiteralQuoteString("'")
+            .withLiteralEscapedQuoteString("''").withUnquotedCasing(Casing.UNCHANGED)
+            .withQuotedCasing(Casing.UNCHANGED).withCaseSensitive(true);
     public static final Context KYUUBI_CONTEXT =
             SqlDialect.EMPTY_CONTEXT.withDatabaseProduct(DatabaseProduct.BIG_QUERY)
                     .withLiteralQuoteString("'").withIdentifierQuoteString("`")

common/src/main/java/com/tencent/supersonic/common/pojo/enums/DataTypeEnums.java

@@ -11,6 +11,8 @@ public enum DataTypeEnums {
 
     DATE("DATE"),
 
+    TIMESTAMP("TIMESTAMP"),
+
     BIGINT("BIGINT"),
 
     INT("INT"),
@@ -39,6 +41,9 @@ public enum DataTypeEnums {
                 return typeEnum;
             }
         }
+        if (type != null && type.toUpperCase().contains("INT")) {
+            return DataTypeEnums.INT;
+        }
         return DataTypeEnums.UNKNOWN;
     }
 }

headless/chat/src/main/java/com/tencent/supersonic/headless/chat/ChatQueryContext.java

@@ -56,7 +56,8 @@ public class ChatQueryContext implements Serializable {
         candidateQueries = candidateQueries.stream()
                 .sorted(Comparator.comparing(
                         semanticQuery -> semanticQuery.getParseInfo().getScore(),
-                        Comparator.reverseOrder())).collect(Collectors.toList());
+                        Comparator.reverseOrder()))
+                .collect(Collectors.toList());
         return candidateQueries;
     }
 

headless/chat/src/main/java/com/tencent/supersonic/headless/chat/mapper/HanlpDictMatchStrategy.java

@@ -9,6 +9,7 @@ import org.springframework.beans.factory.annotation.Autowired;
 import org.springframework.stereotype.Service;
 
 import java.util.ArrayList;
+import java.util.Comparator;
 import java.util.LinkedHashSet;
 import java.util.List;
 import java.util.Set;
@@ -50,8 +51,12 @@ public class HanlpDictMatchStrategy extends SingleMatchStrategy<HanlpMapResult>
             return new ArrayList<>();
         }
         // step3. merge pre/suffix result
+        // sort by similarity (desc) first, then name length (desc), so that
+        // higher-similarity records are inserted first and survive LinkedHashSet dedup
         hanlpMapResults = hanlpMapResults.stream()
-                .sorted((a, b) -> -(b.getName().length() - a.getName().length()))
+                .sorted(Comparator.comparingDouble(HanlpMapResult::getSimilarity).reversed()
+                        .thenComparing((a, b) -> Integer.compare(b.getName().length(),
+                                a.getName().length())))
                 .collect(Collectors.toCollection(LinkedHashSet::new));
 
         // step4. filter by similarity

headless/chat/src/main/java/com/tencent/supersonic/headless/chat/mapper/MapFilter.java

@@ -123,15 +123,9 @@ public class MapFilter {
                     .filter(SchemaElementMatch::isFullMatched).collect(Collectors.toList());
 
             if (!fullMatches.isEmpty()) {
-                // If there are objects with similarity=1.0, choose the one with the longest
-                // detectWord and smallest offset
-                SchemaElementMatch bestMatch = fullMatches.stream()
-                        .max(Comparator.comparing(
-                                (SchemaElementMatch match) -> match.getDetectWord().length()))
-                        .orElse(null);
-                if (bestMatch != null) {
-                    result.add(bestMatch);
-                }
+                // Keep all records with similarity=1.0, as they may correspond to different
+                // elementIds with the same detectWord
+                result.addAll(fullMatches);
             } else {
                 // If there are no objects with similarity=1.0, keep all objects with similarity<1.0
                 result.addAll(group);

headless/server/src/main/java/com/tencent/supersonic/headless/server/service/impl/DataSetServiceImpl.java

@@ -286,10 +286,8 @@ public class DataSetServiceImpl extends ServiceImpl<DataSetDOMapper, DataSetDO>
             return;
         }
 
-        Set<Long> includeAllModelIds = configs.stream()
-                .filter(DataSetModelConfig::getIncludesAll)
-                .map(DataSetModelConfig::getId)
-                .collect(Collectors.toSet());
+        Set<Long> includeAllModelIds = configs.stream().filter(DataSetModelConfig::getIncludesAll)
+                .map(DataSetModelConfig::getId).collect(Collectors.toSet());
 
         if (CollectionUtils.isEmpty(includeAllModelIds)) {
             return;
@@ -302,28 +300,26 @@ public class DataSetServiceImpl extends ServiceImpl<DataSetDOMapper, DataSetDO>
         List<DimensionResp> allDimensions = dimensionService.getDimensions(metaFilter);
         List<MetricResp> allMetrics = metricService.getMetrics(metaFilter);
 
-        Map<Long, List<Long>> modelDimensionMap = allDimensions.stream()
-                .collect(Collectors.groupingBy(
-                        DimensionResp::getModelId,
-                        Collectors.mapping(DimensionResp::getId, Collectors.toList())
-                ));
+        Map<Long, List<Long>> modelDimensionMap =
+                allDimensions.stream().collect(Collectors.groupingBy(DimensionResp::getModelId,
+                        Collectors.mapping(DimensionResp::getId, Collectors.toList())));
 
-        Map<Long, List<Long>> modelMetricMap = allMetrics.stream()
-                .collect(Collectors.groupingBy(
-                        MetricResp::getModelId,
-                        Collectors.mapping(MetricResp::getId, Collectors.toList())
-                ));
+        Map<Long, List<Long>> modelMetricMap =
+                allMetrics.stream().collect(Collectors.groupingBy(MetricResp::getModelId,
+                        Collectors.mapping(MetricResp::getId, Collectors.toList())));
 
         for (DataSetModelConfig config : configs) {
             if (Boolean.TRUE.equals(config.getIncludesAll())) {
                 Long modelId = config.getId();
 
-                List<Long> modelDimensions = modelDimensionMap.getOrDefault(modelId, Lists.newArrayList());
+                List<Long> modelDimensions =
+                        modelDimensionMap.getOrDefault(modelId, Lists.newArrayList());
                 Set<Long> existingDimensions = new HashSet<>(config.getDimensions());
                 existingDimensions.addAll(modelDimensions);
                 config.setDimensions(new ArrayList<>(existingDimensions));
 
-                List<Long> modelMetrics = modelMetricMap.getOrDefault(modelId, Lists.newArrayList());
+                List<Long> modelMetrics =
+                        modelMetricMap.getOrDefault(modelId, Lists.newArrayList());
                 Set<Long> existingMetrics = new HashSet<>(config.getMetrics());
                 existingMetrics.addAll(modelMetrics);
                 config.setMetrics(new ArrayList<>(existingMetrics));

headless/server/src/main/java/com/tencent/supersonic/headless/server/utils/DataSetSchemaBuilder.java

@@ -123,8 +123,10 @@ public class DataSetSchemaBuilder {
                 dimToAdd.getExtInfo().put(DimensionConstants.DIMENSION_DATA_TYPE,
                         dim.getDataType());
             } else {
-                dimToAdd.getExtInfo().put(DimensionConstants.DIMENSION_DATA_TYPE,
-                        dataTypeMap.get(dim.getModelId()).get(dim.getBizName()));
+                if (dataTypeMap.containsKey(dim.getModelId())) {
+                    dimToAdd.getExtInfo().put(DimensionConstants.DIMENSION_DATA_TYPE,
+                            dataTypeMap.get(dim.getModelId()).get(dim.getBizName()));
+                }
             }
             if (dim.isTimeDimension()) {
                 String timeFormat =

headless/server/src/main/java/com/tencent/supersonic/headless/server/utils/DimensionConverter.java

@@ -73,9 +73,6 @@ public class DimensionConverter {
         if (dimensionReq.getTypeParams() != null) {
             dimensionDO.setTypeParams(JSONObject.toJSONString(dimensionReq.getTypeParams()));
         }
-        if (dimensionReq.getDataType() != null) {
-            dimensionDO.setDataType(dimensionReq.getDataType().getType());
-        }
         dimensionDO.setStatus(StatusEnum.ONLINE.getCode());
         return dimensionDO;
     }

headless/server/src/main/resources/mapper/custom/DimensionDOCustomMapper.xml

@@ -11,6 +11,7 @@
         <result column="status" jdbcType="INTEGER" property="status"/>
         <result column="sensitive_level" jdbcType="INTEGER" property="sensitiveLevel"/>
         <result column="type" jdbcType="VARCHAR" property="type"/>
+        <result column="data_type" jdbcType="VARCHAR" property="dataType"/>
         <result column="created_at" jdbcType="TIMESTAMP" property="createdAt"/>
         <result column="created_by" jdbcType="VARCHAR" property="createdBy"/>
         <result column="updated_at" jdbcType="TIMESTAMP" property="updatedAt"/>
@@ -67,14 +68,16 @@
     <insert id="batchInsert" parameterType="java.util.List" useGeneratedKeys="true" keyProperty="id">
         insert into s2_dimension (name, biz_name,
         description, status, model_id,
-        type, type_params, expr,created_at, created_by,
+        type, data_type, type_params, expr,created_at, created_by,
         updated_by, updated_at, semantic_type,sensitive_level, is_tag, ext)
         values
         <foreach collection="list" item="dimension" separator=",">
             (#{dimension.name,jdbcType=VARCHAR}, #{dimension.bizName,jdbcType=VARCHAR},
             #{dimension.description,jdbcType=VARCHAR}, #{dimension.status,jdbcType=INTEGER},
             #{dimension.modelId,jdbcType=BIGINT},
-            #{dimension.type,jdbcType=VARCHAR}, #{dimension.typeParams,jdbcType=VARCHAR},
+            #{dimension.type,jdbcType=VARCHAR},
+            #{dimension.dataType,jdbcType=VARCHAR},
+            #{dimension.typeParams,jdbcType=VARCHAR},
             #{dimension.expr,jdbcType=VARCHAR}, #{dimension.createdAt,jdbcType=TIMESTAMP},
             #{dimension.createdBy,jdbcType=VARCHAR},
             #{dimension.updatedBy,jdbcType=VARCHAR}, #{dimension.updatedAt,jdbcType=TIMESTAMP},
@@ -98,6 +101,9 @@
             <if test="dimension.status != null">status = #{dimension.status,jdbcType=INTEGER},</if>
             <if test="dimension.modelId != null">model_id = #{dimension.modelId,jdbcType=BIGINT},</if>
             <if test="dimension.type != null and dimension.type !=''">type = #{dimension.type,jdbcType=VARCHAR},</if>
+            <if test="dimension.dataType != null and dimension.dataType !=''">data_type =
+                #{dimension.dataType,jdbcType=VARCHAR},
+            </if>
             <if test="dimension.typeParams != null and dimension.typeParams !=''">type_params =
                 #{dimension.typeParams,jdbcType=VARCHAR},
             </if>

webapp/packages/supersonic-fe/src/pages/SemanticModel/Datasource/components/ModelFieldForm.tsx

@@ -105,7 +105,7 @@ const ModelFieldForm: React.FC<Props> = ({
                 let defaultParams:any = {};
                 if (value === EnumDataSourceType.MEASURES) {
                   defaultParams = {
-                    agg: AGG_OPTIONS[0].value,
+                    agg: AGG_OPTIONS[AGG_OPTIONS.length - 1].value,
                     classType: EnumDataSourceType.MEASURES,
                     type: EnumDataSourceType.MEASURES,
                   };
@@ -217,7 +217,7 @@ const ModelFieldForm: React.FC<Props> = ({
                 handleFieldChange(record, 'agg', value);
               }}
               allowClear
-              defaultValue={AGG_OPTIONS[0].value}
+              defaultValue={AGG_OPTIONS[AGG_OPTIONS.length - 1].value}
               style={{ width: '100%' }}
             >
               {AGG_OPTIONS.map((item) => (