codelipenghui commented on a change in pull request #9490:
URL: https://github.com/apache/pulsar/pull/9490#discussion_r584747593



##########
File path: pulsar-common/src/main/proto/PulsarApi.proto
##########
@@ -224,6 +224,7 @@ enum ServerError {
                          // use this error to indicate that this producer is 
now permanently
                          // fenced. Applications are now supposed to close it 
and create a
                          // new producer
+    TransactionBufferNotRecover = 26; // When transaction buffer not recover 
completely

Review comment:
       Shall we need to expose this error to the client-side? It's better to 
handle it in the broker internal.




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


Reply via email to