alamb commented on code in PR #14669: URL: https://github.com/apache/datafusion/pull/14669#discussion_r1957306599
########## datafusion/expr/src/planner.rs: ########## @@ -49,11 +54,19 @@ pub trait ContextProvider { not_impl_err!("Table Functions are not supported") } - /// This provides a worktable (an intermediate table that is used to store the results of a CTE during execution) - /// We don't directly implement this in the logical plan's ['SqlToRel`] - /// because the sql code needs access to a table that contains execution-related types that can't be a direct dependency - /// of the sql crate (namely, the `CteWorktable`). + /// Provides an intermediate table that is used to store the results of a CTE during execution) + /// + /// CTE stands for "Common Table Expression" + /// + /// # Notes + /// We don't directly implement this in ['SqlToRel`] as implementing this function Review Comment: fixed in 54fac30cd ![Uploading Screenshot 2025-02-16 at 6.44.37 AM.png…]() -- 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