kezhenxu94 commented on a change in pull request #4344: supprt gRPC alarm hook.
URL: https://github.com/apache/skywalking/pull/4344#discussion_r377670357
 
 

 ##########
 File path: docs/en/setup/backend/backend-alarm.md
 ##########
 @@ -113,6 +114,24 @@ Example as following
 }]
 ```
 
+## gRPCHook
+The alarm message will send through remote gRPC method by `Protobuf` content 
type. 
+The message format with following key information which are defined in 
`oap-server/server-alarm-plugin/src/main/proto/alarm-hook.proto`.
 
 Review comment:
   We may not need to show "Part of protocol" file, my suggestion is to 
**link** to the `alarm-hook.proto` file directly, because the gRPC receiver may 
be implemented in any language, and the `proto` file is fundamental in any kind 
of implementation. If I were the user, I'd like to click the link and navigate 
to the `proto` file directly, instead of finding in the souce codes myself.

----------------------------------------------------------------
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:
us...@infra.apache.org


With regards,
Apache Git Services

Reply via email to