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

Posted to github@arrow.apache.org by "arthurpassos (via GitHub)" <gi...@apache.org> on 2023/05/31 16:07:19 UTC

[GitHub] [arrow] arthurpassos commented on pull request #35825: GH-32723: [C++][Parquet] Add option to use LARGE* variants of binary types

arthurpassos commented on PR #35825:
URL: https://github.com/apache/arrow/pull/35825#issuecomment-1570518777

   > The skeleton is ok here, but I think too much duplicated code is introduced. Would you mind using template to simplify the code?
   
   Sure, I'll first add some tests and then look into this. If you have any suggestions on what code you would like to be templated, pls let me know


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