clintropolis commented on code in PR #12708:
URL: https://github.com/apache/druid/pull/12708#discussion_r920698691
##########
sql/src/main/java/org/apache/druid/sql/calcite/planner/DruidPlanner.java:
##########
@@ -172,18 +171,19 @@ public ValidationResult validate(boolean
authorizeContextParams) throws SqlParse
* Prepare an SQL query for execution, including some initial parsing and
validation and any dynamic parameter type
* resolution, to support prepared statements via JDBC.
*
- * In some future this could perhaps re-use some of the work done by {@link
#validate(boolean)}
+ * In some future this could perhaps re-use some work done by {@link
#validate(boolean)}
* instead of repeating it, but that day is not today.
*/
+ // RelConversionException is no longer thrown, but removing it causes
Review Comment:
got it, was curious because on the surface it only looks like `SqlLifecycle`
and some tests. As long as we don't forget about it i think its fine.
--
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]
