nicoloboschi commented on code in PR #18898:
URL: https://github.com/apache/pulsar/pull/18898#discussion_r1047643177
##########
pulsar-broker/src/main/java/org/apache/pulsar/broker/service/Producer.java:
##########
@@ -800,7 +800,9 @@ public void checkEncryption() {
public void publishTxnMessage(TxnID txnID, long producerId, long
sequenceId, long highSequenceId,
ByteBuf headersAndPayload, long batchSize,
boolean isChunked, boolean isMarker) {
- checkAndStartPublish(producerId, sequenceId, headersAndPayload,
batchSize, null);
+ if (!checkAndStartPublish(producerId, sequenceId, headersAndPayload,
batchSize, null)) {
+ return;
+ }
Review Comment:
yes it's a bit out of the encryption fix scope. I can open another pull
request
--
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]
