xiangfu0 commented on pull request #8158: URL: https://github.com/apache/pinot/pull/8158#issuecomment-1036838289
Seems Calcite changed the parsed object structure. Can you apply the below change? ``` diff --git a/pinot-common/src/main/java/org/apache/pinot/sql/parsers/CalciteSqlParser.java b/pinot-common/src/main/java/org/apache/pinot/sql/parsers/CalciteSqlParser.java index 03c4f60062..49a67f5680 100644 --- a/pinot-common/src/main/java/org/apache/pinot/sql/parsers/CalciteSqlParser.java +++ b/pinot-common/src/main/java/org/apache/pinot/sql/parsers/CalciteSqlParser.java @@ -711,6 +711,10 @@ public class CalciteSqlParser { compilePathExpression(functionName, functionNode, path); return RequestUtils.getIdentifierExpression(path.toString()); } + if ((functionNode.getFunctionQuantifier() != null) && ("DISTINCT".equals( + functionNode.getFunctionQuantifier().toString()))) { + functionName = "DISTINCTCOUNT"; + } break; default: functionName = functionKind.name(); ``` -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org For additional commands, e-mail: commits-h...@pinot.apache.org