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

Posted to dev@parquet.apache.org by GitBox <gi...@apache.org> on 2020/09/23 17:00:53 UTC

[GitHub] [parquet-mr] SinghAsDev commented on pull request #222: Parquet-313: Implement 3 level list writing rule for Parquet-Thrift

SinghAsDev commented on pull request #222:
URL: https://github.com/apache/parquet-mr/pull/222#issuecomment-697699892


   > @ttim and @tlazaro, do you mind taking a look and letting us know if this looks ok to you? We'd want to get this merged.
   
   Hey @ttim and @tlazaro, we have been using this at Pinterest for a few years just fine, and are wanting to have this merged in so that we can get rid of our internal fork. Also, this is config controlled, so default behavior remains unchanged. Would really appreciate if you can help reviewing this from twitter side, so that we can go ahead with merging this. I can rebase the patch right after.


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