[
https://issues.apache.org/jira/browse/PARQUET-1885?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17163448#comment-17163448
]
ASF GitHub Bot commented on PARQUET-1885:
-----------------------------------------
mauliksoneji commented on pull request #802:
URL: https://github.com/apache/parquet-mr/pull/802#issuecomment-662955559
Also, [this previously merged
commit](https://github.com/apache/parquet-mr/commit/fadbe6ef326faa5984d4d8d6df581a91c8c6cca2)
was throwing an error because of unimported class. It has been fixed in my PR
----------------------------------------------------------------
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)