[jira] [Commented] (ARROW-6276) [C++] Add operator[] to some concrete Array implementations?

["Pradeep Garigipati (Jira)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/ARROW-6276?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17442819#comment-17442819 ] 

Pradeep Garigipati commented on ARROW-6276:
-------------------------------------------

[~wesm] [~apitrou] Is the following kind of change that is intended ? or the operator overload should take a scalar of NumericType ?


{code:java}
bool BooleanArray::operator[](int64_t i) const { return GetView(i); }
value_type NumericArray::operator[](int64_t i) const { return GetView(i); }
TypeClass::DayMilliseconds DayTimeIntervalArray::operator[](int64_t i) const { return GetView(i); }
TypeClass::MonthDayNanos MonthDayNanoIntervalArray::operator[](int64_t i) const { return GetView(i); }{code}

> [C++] Add operator[] to some concrete Array implementations?
> ------------------------------------------------------------
>
>                 Key: ARROW-6276
>                 URL: https://issues.apache.org/jira/browse/ARROW-6276
>             Project: Apache Arrow
>          Issue Type: Improvement
>          Components: C++
>            Reporter: Wes McKinney
>            Assignee: Pradeep Garigipati
>            Priority: Major
>              Labels: good-first-issue
>
> I have occasionally been in situations where I want to pass in e.g. {{Int32Array}} or {{BooleanArray}} into a template that also can accept STL sequence types like {{std::vector<bool>}} -- in such cases the main API of interest is {{[]}}. Would there be interest in aliasing {{operator[]}} to return the result of {{GetView}} (or to deprecate {{GetView}} in favor of {{operator[]}})?



--
This message was sent by Atlassian Jira
(v8.20.1#820001)