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/04/05 08:56:42 UTC
[GitHub] [arrow] amol- commented on a diff in pull request #12765: ARROW-15523: [Python] Support for Datasets as inputs of Joins
amol- commented on code in PR #12765:
URL: https://github.com/apache/arrow/pull/12765#discussion_r842540434
##########
python/pyarrow/_exec_plan.pyx:
##########
@@ -61,10 +65,18 @@ cdef execplan(inputs, output_type, vector[CDeclaration] plan, c_bool use_threads
CTable* c_table
shared_ptr[CTable] c_out_table
shared_ptr[CSourceNodeOptions] c_sourceopts
+ shared_ptr[CScanNodeOptions] c_scanopts
+ shared_ptr[CExecNodeOptions] c_input_node_opts
shared_ptr[CSinkNodeOptions] c_sinkopts
shared_ptr[CAsyncExecBatchGenerator] c_async_exec_batch_gen
shared_ptr[CRecordBatchReader] c_recordbatchreader
vector[CDeclaration].iterator plan_iter
+ vector[CDeclaration.Input] no_c_inputs
+
+ global _dataset_support_initialised
+ if not _dataset_support_initialised:
+ Initialize() # Initialise support for Datasets in ExecPlan
+ _dataset_support_initialised = True
Review Comment:
Good point, I'll change that to be at module level.
--
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