alamb commented on PR #11516: URL: https://github.com/apache/datafusion/pull/11516#issuecomment-2248884474
> @findepi also mentioned `Session` as an option which seems reasonable to me too I slightly prefer `SessionProvider` as I think it is more consistent with the naming conventions of other traits in DataFusion such as `TableProvider`, `CatalogProvider`, `SchemaProvider` etc, (however, there are plenty of counter examples like `ExprSchema`) However, I don't feel super strongly and would be interested in opinions from others if we want to have a `Provider` on the end of the (now named) `Session` trait -- 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
