topdog/supersonic
1
0
Fork 0
mirror of https://github.com/tencentmusic/supersonic.git synced 2025-12-12 12:37:55 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

[improvement][headless]Add more checks to accept() of QueryParsers.

Browse Source
This commit is contained in:
jerryjzhang
2024-12-27 11:57:23 +08:00
parent ade03627ce
commit 920d8f280a
3 changed files with 8 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
headless/core/src/main/java/com/tencent/supersonic/headless/core/translator/parser/DefaultDimValueParser.java
View File

@@ -31,7 +31,8 @@ public class DefaultDimValueParser implements QueryParser {
@Override @Override
public boolean accept(QueryStatement queryStatement) { public boolean accept(QueryStatement queryStatement) {
return Objects.nonNull(queryStatement.getSqlQuery()) return Objects.nonNull(queryStatement.getSqlQuery())
&& StringUtils.isNotBlank(queryStatement.getSqlQuery().getSql()); && StringUtils.isNotBlank(queryStatement.getSqlQuery().getSql())
&& !CollectionUtils.isEmpty(queryStatement.getOntology().getDimensions());
} }
@Override @Override

4
headless/core/src/main/java/com/tencent/supersonic/headless/core/translator/parser/DimExpressionParser.java
View File

@@ -26,7 +26,9 @@ public class DimExpressionParser implements QueryParser {
@Override @Override
public boolean accept(QueryStatement queryStatement) { public boolean accept(QueryStatement queryStatement) {
return Objects.nonNull(queryStatement.getSqlQuery()) return Objects.nonNull(queryStatement.getSqlQuery())
&& StringUtils.isNotBlank(queryStatement.getSqlQuery().getSql()); && Objects.nonNull(queryStatement.getOntologyQuery())
&& StringUtils.isNotBlank(queryStatement.getSqlQuery().getSql())
&& !CollectionUtils.isEmpty(queryStatement.getOntologyQuery().getDimensions());
} }
@Override @Override

4
headless/core/src/main/java/com/tencent/supersonic/headless/core/translator/parser/MetricExpressionParser.java
View File

@@ -25,7 +25,9 @@ public class MetricExpressionParser implements QueryParser {
@Override @Override
public boolean accept(QueryStatement queryStatement) { public boolean accept(QueryStatement queryStatement) {
return Objects.nonNull(queryStatement.getSqlQuery()) return Objects.nonNull(queryStatement.getSqlQuery())
&& StringUtils.isNotBlank(queryStatement.getSqlQuery().getSql()); && Objects.nonNull(queryStatement.getOntologyQuery())
&& StringUtils.isNotBlank(queryStatement.getSqlQuery().getSql())
&& !CollectionUtils.isEmpty(queryStatement.getOntologyQuery().getMetrics());
} }
@Override @Override