github-advanced-security[bot] commented on code in PR #17894:
URL: https://github.com/apache/druid/pull/17894#discussion_r2048572959
##########
sql/src/main/java/org/apache/druid/sql/calcite/planner/DruidPlanner.java:
##########
@@ -283,6 +293,66 @@
planner.close();
}
+ /**
+ * If an {@link SqlNode} is a {@link SqlNodeList}, it must consist of 0 or
more {@link SqlSetOption} followed by a
+ * single {@link SqlNode} which is NOT a {@link SqlSetOption}. All {@link
SqlSetOption} will be converted into a
+ * context parameters {@link Map} and added to the {@link PlannerContext}
with
+ * {@link PlannerContext#addAllToQueryContext(Map)}. The final {@link
SqlNode} of the {@link SqlNodeList} is returned
+ * by this method as the {@link SqlNode} which should actually be validated
and executed, and will have access to the
+ * modified query context through the {@link PlannerContext}. {@link
SqlSetOption} override any existing query
+ * context parameter values.
+ */
+ private SqlNode processStatementList(SqlNode root, String sql)
Review Comment:
## Useless parameter
The parameter 'sql' is never used.
[Show more
details](https://github.com/apache/druid/security/code-scanning/8915)
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
