[rocketmq-apis] branch v2 updated: Add RPC-level status field

[li...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

lizhanhui pushed a commit to branch v2
in repository https://gitbox.apache.org/repos/asf/rocketmq-apis.git


The following commit(s) were added to refs/heads/v2 by this push:
     new 1d9556f  Add RPC-level status field
1d9556f is described below

commit 1d9556fee4427482210b5b7c15b39e66f51ab4dc
Author: Li Zhanhui <li...@gmail.com>
AuthorDate: Tue Apr 19 08:36:50 2022 +0000

    Add RPC-level status field
---
 apache/rocketmq/v2/service.proto | 13 ++++++++++++-
 1 file changed, 12 insertions(+), 1 deletion(-)

diff --git a/apache/rocketmq/v2/service.proto b/apache/rocketmq/v2/service.proto
index 5c5c8d1..c180515 100644
--- a/apache/rocketmq/v2/service.proto
+++ b/apache/rocketmq/v2/service.proto
@@ -104,10 +104,21 @@ message AckMessageRequest {
 message AckMessageResultEntry {
   string message_id = 1;
   string receipt_handle = 2;
+
+  // Acknowledge result may be acquired through inspecting
+  // `status.code`; In case acknowledgement failed, `status.message`
+  // is the explanation of for the failure.
   Status status = 3;
 }
 
-message AckMessageResponse { repeated AckMessageResultEntry entries = 1; }
+message AckMessageResponse {
+
+  // RPC tier status, which is used to represent RPC-level errors including
+  // authentication, authorization, throttling and other general failures.
+  Status status = 1;
+
+  repeated AckMessageResultEntry entries = 2;
+}
 
 message NackMessageRequest {
   Resource group = 1;