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/07/27 10:15:13 UTC

[GitHub] [parquet-mr] gszadovszky commented on pull request #702: PARQUET-1684: dont store default protobuf values as null for proto3

gszadovszky commented on pull request #702:
URL: https://github.com/apache/parquet-mr/pull/702#issuecomment-664263943


   @dossett,
   Strictly speaking this is not a regression in 1.11.0 so not required for 1.11.1. I am not an expert of protobuf (neither in parquet-protobuf) so let me ask the following questions:
   - Do we have a workaround for this issue?
   - What do we think about the potential regression this change may cause? (1.11.1 shall be 100% compatible with 1.11.0 and should not contain any regressions only bugfixes that breaks the usage of 1.11.0.)


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