[GitHub] [arrow-rs] Jimexist opened a new issue #437: migrate partition kernel to use IntoIter trait

[GitBox <gi...@apache.org>]

Jimexist opened a new issue #437:
URL: https://github.com/apache/arrow-rs/issues/437


   It should be possible to write the below as an iterator, thereby avoiding the allocation of the vector.
   
   The general idea: create a struct with a method  `try_new`. That method initializes the struct's content and errors if something is wrong (e.g. empty columns). It also initializes `lexicographical_comparator`.
   
   Next, implement `Iterator<Item=usize>` for that struct where `Iterator::next` yields `previous_partition_point` and increments it as written in this function.
   
   _Originally posted by @jorgecarleitao in https://github.com/apache/arrow-rs/pull/424#discussion_r647625641_


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [arrow-rs] alamb closed issue #437: migrate partition kernel to use Iterator trait

[GitBox <gi...@apache.org>]

alamb closed issue #437:
URL: https://github.com/apache/arrow-rs/issues/437


   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org