tibrewalpratik17 commented on code in PR #10873:
URL: https://github.com/apache/pinot/pull/10873#discussion_r1237089197
##########
pinot-controller/src/main/java/org/apache/pinot/controller/api/resources/PinotQueryResource.java:
##########
@@ -185,8 +190,10 @@ private String getMultiStageQueryResponse(String query,
String queryOptions, Htt
return QueryException.ACCESS_DENIED_ERROR.toString();
}
- SqlNodeAndOptions sqlNodeAndOptions = RequestUtils.parseQuery(query);
- List<String> tableNames =
CalciteSqlParser.extractTableNamesFromNode(sqlNodeAndOptions.getSqlNode());
Review Comment:
Hey @walterddr raised this #10955 to make the change in Pinot client to keep
it independent of this change. Will remove the Calcite-util in this PR or the
other one which one will merge later. Hope so it's okay!
Looking into the integration test failure of this one meanwhile.
--
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
