alamb commented on PR #11516: URL: https://github.com/apache/datafusion/pull/11516#issuecomment-2248817038
> Looking at this with fresh eyes. Is the CatalogSession name the best name we can have? I thought about this a bit. How about `SessionStateProvider`? That follows convention of other `Provider` traits in DataFusion, eg https://docs.rs/datafusion/latest/datafusion/index.html?search=provider, and is makes it pretty clear that this is connected to `SessionState` Here are some other options: * `PlanningState` or `SessionPlanningState` - the idea being this structure contains the information needed to plan the query * `SessionPlanningState` -- 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
