julianhyde commented on code in PR #3984:
URL: https://github.com/apache/calcite/pull/3984#discussion_r1801844549
##########
core/src/main/java/org/apache/calcite/sql/validate/SqlValidatorImpl.java:
##########
@@ -4185,6 +4217,48 @@ private static void forEachQualified(SqlNode node,
SqlValidatorScope scope,
});
}
+ /* If the supplied SqlNode when fully qualified is in the set of
bypassQualifieds, then we
+ remove all entries in the qualifieds set as well as remnantMustFilterFields
Review Comment:
No, it should be a javadoc comment. Also it should be written like a method
javadoc, e.g. "Removes all entries..."
--
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]
