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

Posted to github@arrow.apache.org by GitBox <gi...@apache.org> on 2021/05/30 09:14:32 UTC

[GitHub] [arrow-rs] ritchie46 commented on pull request #382: make sure that only concat preallocates buffers

ritchie46 commented on pull request #382:
URL: https://github.com/apache/arrow-rs/pull/382#issuecomment-850967804


   @jorgecarleitao I implemented your proposal from #347. As we now need to define an `enum` in the `with_capacities`  constructor, I want to be extra certain that I added all possible variants, as adding one later is backwards incompatable. Could you check if I missed something.
   
   For now, only the `(large)-utf8` ones uses this, but if the design is ok, I can add more in later PRs.


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