[C++] Attach arrow::ArrayStatistics to arrow::Array
#43666
Assignees
Labels
Comments
kou
added a commit
to kou/arrow
that referenced
this issue
Aug 15, 2024
New public APIs: * `arrow::Array::statistics()`: It returns the associated statistics. It can't be changed after an array is created. A sliced array doesn't have parent array's statistics because parent array's statistics isn't valid for sliced array. * Add new optional `arrow::ArrayStatistics` argument to all `arrow::*Array(ArrayData)` constructors: `arrow::*Array(ArrayData, ArrayStatistics = NULLPTR)` New internal APIs: * `arrow::Array::Init()`: All array constructors must call this to attach `arrow::ArrayData` and `arrow::ArrayStatistics`. Note that calling this via parent's constructor isn't allowed. Array constructors don't need to call `arrow::Array::SetData()` directly. It's called in `arrow::Array::Init()`. * `arrow::Array::SetStatitics()`: It attaches `arrow::ArrayStatistics` to `arrow::Array`. In general, this is not called directly. This is called from `arrow::Array::Init()` internally. Changed internal APIs: * `arrow::Array::SetData()`: It becomes a virtual method. So `arrow::Array::Init()` must be called by each array's constructor.
|
We've attached to |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Describe the enhancement requested
GH-41909 introduced
arrow::ArrayStatitsticsbut it's not associated witharrow::Array.This issue attaches
arrow::ArrayStatisticstoarrow::Array.See also:
Component(s)
C++
The text was updated successfully, but these errors were encountered: