alamb commented on code in PR #14669:
URL: https://github.com/apache/datafusion/pull/14669#discussion_r1956363618
##########
datafusion/core/src/lib.rs:
##########
@@ -229,9 +229,9 @@
//! 1. The query string is parsed to an Abstract Syntax Tree (AST)
//! [`Statement`] using [sqlparser].
//!
-//! 2. The AST is converted to a [`LogicalPlan`] and logical
-//! expressions [`Expr`]s to compute the desired result by the
-//! [`SqlToRel`] planner.
+//! 2. The AST is converted to a [`LogicalPlan`] and logical expressions
+//! [`Expr`]s to compute the desired result by [`SqlToRel`]. This phase
+//! also includes name and type resolution ("binding").
Review Comment:
I mostly wanted the term "binding" to appear here, but then I got carried
away when I explored a bit more
--
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: github-unsubscr...@datafusion.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org
For additional commands, e-mail: github-h...@datafusion.apache.org
