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

Reply via email to