You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@arrow.apache.org by "felipecrv (via GitHub)" <gi...@apache.org> on 2024/06/05 00:34:56 UTC

[I] [C++] Miscellaneous fixes to make supporting list-view where lists are supported a bit easier [arrow]

felipecrv opened a new issue, #41970:
URL: https://github.com/apache/arrow/issues/41970

   ### Describe the enhancement requested
   
   Issue to track multiple small issues I fixed while working on a bigger PR expanding kernels to support list-view types as well.
    
    - Inclusion of `FixedSizeListBuilder::TypeClass` to make it in line with `VarLengthListLikeBuilder<TYPE>` classes
    - Update documentation where list-view types should be mentioned
    - Change `list`/`large_list`/`fixed_size_list` factories to take `shared_ptr` by value instead of `const &` the same way `list_view` and `large_list_view`
    
   The last one allows unit tests to have an array of factory function that take the `std::shared_ptr<DataType>` parameter by value.
   
   ### Component(s)
   
   C++


-- 
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: issues-unsubscribe@arrow.apache.org.apache.org

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

Re: [I] [C++] Miscellaneous fixes to make supporting list-view where lists are supported a bit easier [arrow]

Posted by "felipecrv (via GitHub)" <gi...@apache.org>.

felipecrv closed issue #41970: [C++] Miscellaneous fixes to make supporting list-view where lists are supported a bit easier
URL: https://github.com/apache/arrow/issues/41970


-- 
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: issues-unsubscribe@arrow.apache.org

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

Re: [I] [C++] Miscellaneous fixes to make supporting list-view where lists are supported a bit easier [arrow]

Posted by "felipecrv (via GitHub)" <gi...@apache.org>.

felipecrv commented on issue #41970:
URL: https://github.com/apache/arrow/issues/41970#issuecomment-2158568135

   Issue resolved by pull request 41971
   https://github.com/apache/arrow/pull/41971


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