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/11/18 16:59:02 UTC

[GitHub] [arrow-datafusion-python] andygrove commented on a diff in pull request #76: allow config options to be passed to context

andygrove commented on code in PR #76:
URL: https://github.com/apache/arrow-datafusion-python/pull/76#discussion_r1026667441


##########
src/context.rs:
##########
@@ -75,9 +79,23 @@ impl PySessionContext {
         repartition_windows: bool,
         parquet_pruning: bool,
         target_partitions: Option<usize>,
-        // TODO: config_options
+        config_options: Option<HashMap<String, String>>,

Review Comment:
   Yes, for now .. I wasn't sure how to do it in a type-safe way with Python. It would be good to look at improving this in a follow on PR.
   
   I tried using BuiltInConfig but the data is private so I could not access it from this project.



-- 
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