lizhanhui commented on code in PR #27:
URL: https://github.com/apache/rocketmq-apis/pull/27#discussion_r860560949


##########
apache/rocketmq/v2/service.proto:
##########
@@ -80,6 +84,8 @@ message ReceiveMessageRequest {
   int32 batch_size = 4;
   // Required if client type is simple consumer.
   optional google.protobuf.Duration invisible_duration = 5;
+  // For message auto renew and clean
+  bool auto_renew = 6;

Review Comment:
   If I am guessing correctly, you are assuming this field is a flag 
instructing proxies to extend the invisible duration of messages on behalf of 
clients. 
   
   It would be kind of strange adding this flag to this RPC. Would it be better 
to add this flag to the receive-message-RPC?



-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to