Skip to content

ARROW-13268: [C++] debugging hash semi-join - #10948

Closed
michalursa wants to merge 28 commits into
apache:mainfrom
michalursa:ARROW-13268-m
Closed

ARROW-13268: [C++] debugging hash semi-join#10948
michalursa wants to merge 28 commits into
apache:mainfrom
michalursa:ARROW-13268-m

Conversation

@michalursa

@michalursamichalursa commented Aug 16, 2021

Copy link
Copy Markdown
Contributor

This PR is for debugging purposes only. Not for review.

@github-actions

Copy link
Copy Markdown

Thanks for opening a pull request!

If this is not a minor PR. Could you open an issue for this pull request on JIRA? https://issues.apache.org/jira/browse/ARROW

Opening JIRAs ahead of time contributes to the Openness of the Apache Arrow project.

Then could you also rename pull request title in the following format?

ARROW-${JIRA_ID}: [${COMPONENT}] ${SUMMARY}

or

MINOR: [${COMPONENT}] ${SUMMARY}

See also:

@michalursamichalursa changed the title Arrow 13268 mARROW-13268: [C++] debugging hash semi-joinAug 17, 2021
@github-actions

Copy link
Copy Markdown

nirandapereraand others added 7 commits August 17, 2021 00:10
… factories to a registry
- An extensible registry of exec node factories (`std::function<Result<ExecNode*>(ExecPlan* plan, std::vector<ExecNode*> inputs, const ExecNodeOptions& options)>`) is provided
- Hard coded factories like `compute::MakeSinkNode`, `dataset::MakeScanNode` are replaced by factories in the registry named "sink", "scan", etc
- `arrow::compute::Declaration` is provided to represent an unconstructed set of `ExecNode`s, which can be validated and emplaced into an `ExecPlan` as a unit
Closesapache#10793 from bkietz/exec-node-factory-registry
Authored-by: Benjamin Kietzman <bengilgit@gmail.com>
Signed-off-by: Benjamin Kietzman <bengilgit@gmail.com>
@pitrou

Copy link
Copy Markdown
Member

@michalursa Do you need to keep this open for further experiments or can this be closed?

@github-actions

Copy link
Copy Markdown

Thank you for your contribution. Unfortunately, this pull request has been marked as stale because it has had no activity in the past 365 days. Please remove the stale label or comment below, or this PR will be closed in 14 days. Feel free to re-open this if it has been closed in error. If you do not have repository permissions to reopen the PR, please tag a maintainer.

@github-actionsgithub-actionsBot added the Status: stale-warning Issues and PRs flagged as stale which are due to be closed if no indication otherwise label Nov 18, 2025
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Component: C++Status: stale-warningIssues and PRs flagged as stale which are due to be closed if no indication otherwise

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

@michalursa@pitrou@ianmcook@nirandaperera@bkietz