pulsar icon indicating copy to clipboard operation
pulsar copied to clipboard

Published 20 hours ago • verified publisher icon apache

[improve][broker] Remove check ack.getMessageIdsCount() == 1 for cumulative ack

Open AnonHxy opened this issue 2 years ago • 0 comments

Motivation

A minor improvement by removing check ack.getMessageIdsCount() == 1 for cumulative ack, making the code more clear and effective. Because we have checked it before at line418

Modifications

Remove check ack.getMessageIdsCount() == 1 for cumulative ack

Verifying this change

  • [x] Make sure that the change passes the CI checks.

Documentation

  • [ ] doc
  • [ ] doc-required
  • [x] doc-not-needed
  • [ ] doc-complete

Matching PR in forked repository

PR in forked repository: https://github.com/AnonHxy/pulsar/pull/14

AnonHxy avatar Nov 04 '22 03:11 AnonHxy