You are viewing a plain text version of this content. The canonical link for it is here.

Posted to github@arrow.apache.org by GitBox <gi...@apache.org> on 2022/05/30 19:25:30 UTC

[GitHub] [arrow-datafusion] tustvold commented on a diff in pull request #2660: Pass SessionState to TableProvider::scan

tustvold commented on code in PR #2660:
URL: https://github.com/apache/arrow-datafusion/pull/2660#discussion_r885045639


##########
datafusion/core/src/execution/context.rs:
##########
@@ -931,6 +931,11 @@ impl SessionContext {
     pub fn task_ctx(&self) -> Arc<TaskContext> {
         Arc::new(TaskContext::from(self))
     }
+
+    /// Get a copy of the [`SessionState`] of this [`SessionContext`]

Review Comment:
   I'm not a massive fan of this, but it appears to be what we do already for every query and when constructing a TaskContext, so it can't be that bad..........



-- 
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-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org