You are viewing a plain text version of this content. The canonical link for it is here.

Posted to notifications@yetus.apache.org by "Allen Wittenauer (Jira)" <ji...@apache.org> on 2021/12/15 08:51:00 UTC

[jira] [Commented] (YETUS-1151) New Yetus check for `.proto` files

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

Allen Wittenauer commented on YETUS-1151:
-----------------------------------------

It is probably worth mentioning but Apache Yetus already has this functionality via buf support.

> New Yetus check for `.proto` files
> ----------------------------------
>
>                 Key: YETUS-1151
>                 URL: https://issues.apache.org/jira/browse/YETUS-1151
>             Project: Yetus
>          Issue Type: Improvement
>            Reporter: Yongkang Li
>            Priority: Major
>
> Regarding some issues caused by the incompatibility of protobuf message, we have created a static checker which keeps track of the proto file change, and detects potential incompatibility:
>  # Add/delete required fields.
>  # The tag number of a field has been changed.
>  # A  required field has been changed to optional, or an optional field has been changed to required.
> We have tested the checker on HBASE and found many potential problems. We created HBASE-25340 (you can find more detailed information in this issue) and maintainers suggest that we could get the tool added as a new Yetus check so that it could be easily integrated into HBase and Hadoop CI testing.
> Repository of our static checker: [https://github.com/YongkangLi/ProtoBufChecker].



--
This message was sent by Atlassian Jira
(v8.20.1#820001)