Skip to content

ARROW-14008: [R][Compute] Running an ExecPlan should yield Reader instead of Table - #11213

Closed
bkietz wants to merge 1 commit into
apache:masterfrom
bkietz:14008-ExecPlan-run-should-retur
Closed

ARROW-14008: [R][Compute] Running an ExecPlan should yield Reader instead of Table#11213
bkietz wants to merge 1 commit into
apache:masterfrom
bkietz:14008-ExecPlan-run-should-retur

Conversation

@bkietz

Copy link
Copy Markdown
Member

This only modifies the binding (c++) so that returning a RecordBatchReader is possible; the R API continues to collect the stream of batches into a Table.

@github-actions

Copy link
Copy Markdown

@nealrichardsonnealrichardson left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@bkietz
bkietz deleted the 14008-ExecPlan-run-should-retur branch September 23, 2021 20:38
ViniciusSouzaRoque pushed a commit to s1mbi0se/arrow that referenced this pull request Oct 20, 2021
…tead of Table
This only modifies the binding (c++) so that returning a RecordBatchReader is possible; the R API continues to collect the stream of batches into a Table.
Closesapache#11213 from bkietz/14008-ExecPlan-run-should-retur
Authored-by: Benjamin Kietzman <bengilgit@gmail.com>
Signed-off-by: Benjamin Kietzman <bengilgit@gmail.com>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@bkietz@nealrichardson