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

["Yongkang Li (Jira)" <ji...@apache.org>]

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

Yongkang Li edited comment on YETUS-1151 at 12/27/21, 2:25 AM:
---------------------------------------------------------------

Hi [~aw]! I see. Buf indeed provides many functionalities. We also have a functionality that buf seems not to provide yet, a checker for enum whose order is used in serialization. If you feel this functionality is valuable to be integrated into buf, we are more than willing to offer any help. 


was (Author: yongkang li):
Hi [~aw]! I see. It seems that buf provides many functionalities. However, we have a functionality that buf does not provide yet, a checker for enum whose order is used in serialization. If you feel this functionality is valuable to be integrated into buf, we are more than willing to offer any help. 

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