[
https://issues.apache.org/jira/browse/PARQUET-2070?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17392786#comment-17392786
]
ASF GitHub Bot commented on PARQUET-2070:
-----------------------------------------
gszadovszky commented on pull request #919:
URL: https://github.com/apache/parquet-mr/pull/919#issuecomment-892415314
@sv3ndk, we have similar failures in Travis for other PRs as well. (Created
INFRA-22171 about this issue.) Travis test is there to cover arm64
architecture. I don't think this change would hurt arm64 compatibility so I'm
pushing this one.
--
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.
To unsubscribe, e-mail: dev-unsubscr...@parquet.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
> Replace deprecated syntax in protobuf support
> ---------------------------------------------
>
> Key: PARQUET-2070
> URL: https://issues.apache.org/jira/browse/PARQUET-2070
> Project: Parquet
> Issue Type: Improvement
> Reporter: Svend Vanderveken
> Priority: Minor
>
> This is trivial change, though at the moment ProtoWriteSupport.java is
> producing a human-readable JSON output of the protobuf schema with the
> following deprecated syntax:
>
> {code:java}
> TextFormat.printToString(asProto){code}
>
> Also, the method where is code is present executed one reflection invocation
> to get the protobuf descriptor which is unnecesserary since the context from
> where it's called already has this descriptor.
> => all minor and trivial stuff though well, housekeeping I guess :)
>
>
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
