Skip to content

Commit 6c6ab23

Browse files
committed
adress comment
1 parent b296601 commit 6c6ab23

File tree

2 files changed

+4
-4
lines changed

2 files changed

+4
-4
lines changed

managed-ledger/src/main/java/org/apache/bookkeeper/mledger/impl/AckSetPositionImpl.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -35,9 +35,9 @@ public class AckSetPositionImpl implements Position, AckSetState {
3535
protected volatile long[] ackSet;
3636
@Getter
3737
@Setter
38-
private int batchMessagesAckedCount;
38+
private volatile int batchMessagesAckedCount;
3939
@Getter
40-
private boolean positionRemovedFromCursor;
40+
private volatile boolean positionRemovedFromCursor;
4141

4242
public AckSetPositionImpl(long ledgerId, long entryId, long[] ackSet) {
4343
this.ledgerId = ledgerId;

pulsar-broker/src/main/java/org/apache/pulsar/broker/service/persistent/PersistentSubscription.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -662,8 +662,8 @@ private void updatePendingAckMessagesAfterAcknowledged0(Consumer ackFrom, List<P
662662
// Reduce un-acknowledged messages.
663663
owner.addAndGetUnAckedMsgs(owner, -actualAcked);
664664
owner.updateBlockedConsumerOnUnackedMsgs(owner);
665-
if (log.isInfoEnabled()) {
666-
log.info("[{}][{}] {}-{}-{} {}-{}-{} acknowledged {} messages, un-ack-msg: {}, position: {}:{}"
665+
if (log.isDebugEnabled()) {
666+
log.debug("[{}][{}] {}-{}-{} {}-{}-{} acknowledged {} messages, un-ack-msg: {}, position: {}:{}"
667667
+ " batch messages acked: {}, position {}:{} was deleted: {}",
668668
topicName, subName, owner.cnx(), owner.consumerId(), owner.consumerName(),
669669
ackFrom == null ? "null" : ackFrom.cnx(),

0 commit comments

Comments
 (0)