apache / apache/arrow

[C++] Add ExecNode to arrow/compute/api.h

Open
#30,757 3 comments 0 reactions 0 assignees View on GitHub
Component: C++ Type: task
Dominant language
C++
Stars
17.1k
Forks
4.3k
Avg merge
3d 13h
Merged PRs (30d)
88

Description

Current, if I want to do anything with exec plans I have to have:

```Java

#include
```

This should not be required if I already have:

```Java

#include
```

Note, when ARROW-15238 is finished the correct file might be something like `arrow/engine/api.h` instead of `arrow/compute/api.h`

**Reporter**: [Weston Pace](https://issues.apache.org/jira/browse/ARROW-15263) / @westonpace

**Note**: *This issue was originally created as [ARROW-15263](https://issues.apache.org/jira/browse/ARROW-15263). Please see the [migration documentation](https://github.com/apache/arrow/issues/14542) for further details.*

Contributor guide

Open the contributing guide

Research direction

Start with arrow/compute/api.h and arrow/compute/exec/exec_plan.h to compare the public API and the current ExecNode location. Check ARROW-15238 and the migration documentation before choosing between arrow/compute/api.h and arrow/engine/api.h; done when exec-plan users can access ExecNode through the intended public header without the extra include.

Written by the indexing model from the issue text.

Assessment

Tech stack
cpp
Domain
backend-api-design
Issue type
Feature
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
48/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.