Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[Python][C++] Add use_threads to run_substrait_query #33212

Closed
asfimport opened this issue Oct 12, 2022 · 0 comments · Fixed by #33623
Closed

[Python][C++] Add use_threads to run_substrait_query #33212

asfimport opened this issue Oct 12, 2022 · 0 comments · Fixed by #33623
Assignees
@westonpace
Labels
Component: C++ Component: Python Type: enhancement
Milestone
11.0.0

Comments

@asfimport
Copy link

Once ARROW-15732 is merged then we should add a boolean use_threads argument to the python run_substrait_query method.

Reporter: Weston Pace / @westonpace
Assignee: Weston Pace / @westonpace

Note: This issue was originally created as ARROW-18008. Please see the migration documentation for further details.
@github-actions github-actions bot mentioned this issue Jan 12, 2023
Merged
westonpace added a commit that referenced this issue Jan 13, 2023
@westonpace @vibhatha
GH-33212: [C++][Python] Add use_threads to pyarrow.substrait.run_query (
e1027dc 
#33623)

Also adds memory_pool and & function_registry to the various DeclarationToXyz methods.  Converts `ExecuteSerializedPlan` to `DeclarationToReader` instead of the bespoke thing it was doing before.
* Closes: #33212

Lead-authored-by: Weston Pace <weston.pace@gmail.com>
Co-authored-by: Vibhatha Lakmal Abeykoon <vibhatha@users.noreply.github.com>
Signed-off-by: Weston Pace <weston.pace@gmail.com>
@westonpace westonpace added this to the 11.0.0 milestone Jan 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@westonpace westonpace

Labels
Component: C++ Component: Python Type: enhancement
Projects
None yet
Milestone
11.0.0
Development

Successfully merging a pull request may close this issue.

GH-33212: [C++][Python] Add use_threads to pyarrow.substrait.run_query westonpace/arrow
2 participants
@westonpace @asfimport