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 2020/10/01 03:10:30 UTC

[GitHub] [arrow] thamht4190 commented on pull request #8023: ARROW-9318: [C++] Parquet encryption key management

thamht4190 commented on pull request #8023:
URL: https://github.com/apache/arrow/pull/8023#issuecomment-701817028


   > In general, this PR is difficult to review due to its size. Would it be possible to break it into a two or more smaller PRs?
   In theory, I think yes, we can create each PR for a group of some classes but it may be hard for reviewers to see an overview. What about keeping this PR to see an overview, I'll create some smaller PRs to review? @ggershinsky
   
   


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