[
https://issues.apache.org/jira/browse/PARQUET-1885?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17163062#comment-17163062
]
ASF GitHub Bot commented on PARQUET-1885:
-----------------------------------------
mauliksoneji commented on pull request #802:
URL: https://github.com/apache/parquet-mr/pull/802#issuecomment-662681612
Hello @Fokko thank you very much for checking the PR.
I have fixed the conflicts and pushed the code with additional tests.
----------------------------------------------------------------
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:
[email protected]
> [parquet-protobuf] Pass descriptor to ProtoWriteSupport constructor
> --------------------------------------------------------------------
>
> Key: PARQUET-1885
> URL: https://issues.apache.org/jira/browse/PARQUET-1885
> Project: Parquet
> Issue Type: Improvement
> Components: parquet-mr
> Affects Versions: 1.11.0, 1.10.1
> Reporter: Maulik Soneji
> Priority: Major
>
> Currently, the ProtoWriteSupport class checks for Descriptor by calling
> `Protobufs.getMessageDescriptor` function which checks for descriptor in the
> classpath. There is no way to pass descriptor as an argument to the
> ProtoWriteSupport constructor.
> In our approach to using parquet-mr library, we are using a descriptor that
> is not available in the classpath.
> I will be happy to work on adding this support to the parquet-mr library.
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
