[GitHub] [arrow] edponce edited a comment on pull request #12055: ARROW-11989: [C++][Python] Improve ChunkedArray's complexity for the access of elements

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

edponce edited a comment on pull request #12055:
URL: https://github.com/apache/arrow/pull/12055#issuecomment-1066464298


   There are 2 main solutions for the type of `chunk_resolver_` in `ChunkedArray`:
   1. Using a `std::unique_ptr` for lazy initialization (valid approach bc `ChunkedArray` is not copyable/assignable). Drawbacks are:
       * the indirect access in the critical path
   2. Using a `ChunkResolver` instance. Drawbacks are:
       * requires defining a default constructor which allows creating empty `ChunkResolver` instances which are not that usable
       * requires defining a move assignment
       * requires defining a default copy constructor
       * making `offsets_` non-const bc assignment is necessary for lazy evaluation
       * a boolean `ChunkedArray` member variable to track lazy initialization


-- 
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.

To unsubscribe, e-mail: github-unsubscribe@arrow.apache.org

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