You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@parquet.apache.org by "ASF GitHub Bot (Jira)" <ji...@apache.org> on 2020/07/20 19:05:00 UTC

[jira] [Commented] (PARQUET-1885) [parquet-protobuf] Pass descriptor to ProtoWriteSupport constructor

    [ https://issues.apache.org/jira/browse/PARQUET-1885?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17161473#comment-17161473 ] 

ASF GitHub Bot commented on PARQUET-1885:
-----------------------------------------

mauliksoneji opened a new pull request #802:
URL: https://github.com/apache/parquet-mr/pull/802


   addresses https://issues.apache.org/jira/browse/PARQUET-1885


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


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