arrow icon indicating copy to clipboard operation
arrow copied to clipboard

[C++] Improve buffer symbolic index

Open asfimport opened this issue 6 years ago • 6 comments

The array data buffers vector is index differently depending on the Array type. This feature would expose static constexpr named variables for buffer index.

Reporter: Francois Saint-Jacques / @fsaintjacques

Note: This issue was originally created as ARROW-4729. Please see the migration documentation for further details.

asfimport avatar Mar 01 '19 16:03 asfimport

Antoine Pitrou / @pitrou: Can you detail what you mean here exactly?

asfimport avatar Jun 11 '19 20:06 asfimport

Wes McKinney / @wesm: probably


static constexpr int kBufferValidity = 0;
static constexpr int kBufferFixedSizeData = 1;
static constexpr int kBufferVarOffsets = 1;
static constexpr int kBufferVarData = 2;

etc

asfimport avatar Jun 11 '19 20:06 asfimport

Francois Saint-Jacques / @fsaintjacques: Precisely

asfimport avatar Jun 12 '19 12:06 asfimport

Antoine Pitrou / @pitrou: Pretty minor, but could be nice to have in 1.0.0.

asfimport avatar Sep 17 '19 16:09 asfimport

Antoine Pitrou / @pitrou: This is probably still nice to have. Should probably be members of the respective DataType subclasses?

asfimport avatar Aug 04 '21 09:08 asfimport

This issue 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 issue will be closed in 14 days. If this improvement is still desired but has no current owner, please add the 'Status: needs champion' label.

github-actions[bot] avatar Dec 14 '25 11:12 github-actions[bot]