mirror of
https://github.com/tencentmusic/supersonic.git
synced 2025-12-16 15:12:26 +00:00
[improvement](chat) Fix get metrics error in queryReqConverter (#547)
This commit is contained in:
@@ -289,6 +289,7 @@ public class SqlParserSelectHelper {
|
|||||||
Set<FieldExpression> orderByFieldExpressions = getOrderByFields(plainSelect);
|
Set<FieldExpression> orderByFieldExpressions = getOrderByFields(plainSelect);
|
||||||
Set<String> collect = orderByFieldExpressions.stream()
|
Set<String> collect = orderByFieldExpressions.stream()
|
||||||
.map(fieldExpression -> fieldExpression.getFieldName())
|
.map(fieldExpression -> fieldExpression.getFieldName())
|
||||||
|
.filter(Objects::nonNull)
|
||||||
.collect(Collectors.toSet());
|
.collect(Collectors.toSet());
|
||||||
result.addAll(collect);
|
result.addAll(collect);
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -170,6 +170,11 @@ class SqlParserSelectHelperTest {
|
|||||||
+ "WHERE MONTH(数据日期) = 9) FROM 营销 WHERE 国家中文名 = '中国' AND MONTH(数据日期) = 9");
|
+ "WHERE MONTH(数据日期) = 9) FROM 营销 WHERE 国家中文名 = '中国' AND MONTH(数据日期) = 9");
|
||||||
|
|
||||||
Assert.assertEquals(allFields.size(), 3);
|
Assert.assertEquals(allFields.size(), 3);
|
||||||
|
|
||||||
|
allFields = SqlParserSelectHelper.getAllFields(
|
||||||
|
"SELECT 用户, 页面 FROM 超音数用户部门 GROUP BY 用户, 页面 ORDER BY count(*) DESC");
|
||||||
|
|
||||||
|
Assert.assertEquals(allFields.size(), 2);
|
||||||
}
|
}
|
||||||
|
|
||||||
@Test
|
@Test
|
||||||
|
|||||||
Reference in New Issue
Block a user