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

[C++] Add a convenient function that converts arrow::ArrayStatistics to arrow::Array for the Arrow C data interface #44010

Closed
kou opened this issue Sep 9, 2024 · 1 comment
Closed

[C++] Add a convenient function that converts arrow::ArrayStatistics to arrow::Array for the Arrow C data interface #44010

kou opened this issue Sep 9, 2024 · 1 comment
Assignees
@kou
Labels
Component: C++ Type: enhancement
Milestone
19.0.0

Comments

@kou
Copy link
Member

kou commented Sep 9, 2024

Describe the enhancement requested

GH-38837 will introduce the schema for statistics arrow::Array that can be interchanged via the Arrow C data interface.

If there is a convenient function for it, it's easy to use GH-38837.

Component(s)

C++
@kou kou changed the title [C++] Add a convenient function that converts arrow::ArrayStatistics to arrow::Array for the Arrpw C data interface [C++] Add a convenient function that converts arrow::ArrayStatistics to arrow::Array for the Arrow C data interface Sep 30, 2024
kou added a commit to kou/arrow that referenced this issue Sep 30, 2024
@kou
apacheGH-44010: [C++] Add RecordBatch::MakeStatisticsArray()
903e3f4 
It's a convenient function that converts `arrow::ArrayStatistics` in a
`arrow::RecordBatch` to `arrow::Array` for the Arrow C data interface.
@github-actions github-actions bot mentioned this issue Sep 30, 2024
Merged
kou added a commit to kou/arrow that referenced this issue Sep 30, 2024
@kou
apacheGH-44010: [C++] Add RecordBatch::MakeStatisticsArray()
92afc83 
It's a convenient function that converts `arrow::ArrayStatistics` in a
`arrow::RecordBatch` to `arrow::Array` for the Arrow C data interface.
kou added a commit to kou/arrow that referenced this issue Sep 30, 2024
@kou
apacheGH-44010: [C++] Add RecordBatch::MakeStatisticsArray()
b194430 
It's a convenient function that converts `arrow::ArrayStatistics` in a
`arrow::RecordBatch` to `arrow::Array` for the Arrow C data interface.
kou added a commit to kou/arrow that referenced this issue Oct 30, 2024
@kou
apacheGH-44010: [C++] Add RecordBatch::MakeStatisticsArray()
5cc013e 
It's a convenient function that converts `arrow::ArrayStatistics` in a
`arrow::RecordBatch` to `arrow::Array` for the Arrow C data interface.
kou added a commit to kou/arrow that referenced this issue Oct 31, 2024
@kou
apacheGH-44010: [C++] Add RecordBatch::MakeStatisticsArray()
089f0a3 
It's a convenient function that converts `arrow::ArrayStatistics` in a
`arrow::RecordBatch` to `arrow::Array` for the Arrow C data interface.
kou added a commit to kou/arrow that referenced this issue Nov 6, 2024
@kou
apacheGH-44010: [C++] Add RecordBatch::MakeStatisticsArray()
1eeb318 
It's a convenient function that converts `arrow::ArrayStatistics` in a
`arrow::RecordBatch` to `arrow::Array` for the Arrow C data interface.
kou added a commit to kou/arrow that referenced this issue Nov 6, 2024
@kou
apacheGH-44010: [C++] Add RecordBatch::MakeStatisticsArray()
0787eeb 
It's a convenient function that converts `arrow::ArrayStatistics` in a
`arrow::RecordBatch` to `arrow::Array` for the Arrow C data interface.
kou added a commit that referenced this issue Nov 8, 2024
@kou
GH-44010: [C++] Add arrow::RecordBatch::MakeStatisticsArray() (#44252)
d748ace 
### Rationale for this change

Statistics schema for Arrow C data interface (GH-43553) is complex because it uses nested types (struct, map and union). So reusable implementation to make statistics array is useful. 

### What changes are included in this PR?

`arrow::RecordBatch::MakeStatisticsArray()` is a convenient function that converts `arrow::ArrayStatistics` in a `arrow::RecordBatch` to `arrow::Array` for the Arrow C data interface.

### Are these changes tested?

Yes.

### Are there any user-facing changes?

Yes.
* GitHub Issue: #44010

Authored-by: Sutou Kouhei <kou@clear-code.com>
Signed-off-by: Sutou Kouhei <kou@clear-code.com>
@kou kou added this to the 19.0.0 milestone Nov 8, 2024
@kou
Copy link
Member Author

kou commented Nov 8, 2024

Issue resolved by pull request 44252
#44252

@kou kou closed this as completed Nov 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@kou kou

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

No branches or pull requests
1 participant
@kou