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/10/02 19:52:22 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-702927968


   > I've run through the changeset and found some minor issues.
   > I am not an expert in parquet-thrift so let's wait for @ttim's approval. I'll push this change if there are no objections.
   
   Thanks @gszadovszky! Appreciate the review. I addressed the feedbacks. Could you give it a final pass and help with merging? I can address @ttim's feedbacks, if any, in a follow up diff.


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