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

Dhruv Vats edited comment on ARROW-13663 at 11/18/21, 9:47 AM:
---------------------------------------------------------------

>From what I understand, to support STL-like iteration, the following operators 
>will have to be overloaded:

++ (increment) (also – ?)

{{*}} (dereference)

!= (inequality)

== (equality)

right?

 

Also, I'm trying to find my way through the code; in this context, could some 
basic guidelines be provided? Like how should this functionality be 
implemented? Whether to use a nested class like in the case of 
{{{}Iterator<T>{}}}, or some other way?

 

Sorry if I'm overlooking something obvious.


was (Author: dhruv9vats):
>From what I understand, to support STL-like iteration, the following operators 
>will have to be overloaded:

++ (increment) (also -- ?)

* (dereference)

!= (inequality)

== (equality)

right?

 

Also, I'm trying to find my way through the code; in this context, could some 
basic guidelines be provided? Like how should this functionality be 
implemented? Whether to use a nested class like in the case of Iterator<T>, or 
some other way?

 

Sorry if I'm overlooking something obvious.

> [C++] RecordBatchReader should support STL-like iteration
> ---------------------------------------------------------
>
>                 Key: ARROW-13663
>                 URL: https://issues.apache.org/jira/browse/ARROW-13663
>             Project: Apache Arrow
>          Issue Type: Improvement
>          Components: C++
>            Reporter: Antoine Pitrou
>            Assignee: Dhruv Vats
>            Priority: Major
>              Labels: beginner, easy, good-first-issue
>             Fix For: 7.0.0
>
>
> Our custom {{Iterator<T>}} has support for STL-like iteration (allowing 
> natural for loops), but {{RecordBatchReader}} doesn't. Adding STL-like 
> iteration would be a significant quality of life improvement for developers



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

Reply via email to