Skip to content

Commit b1698da

Browse files
committed
Move SemaphoreBackPressureHandler#release(amount, reason) implementation to BatchAwareBackPressureHandler interface (#1251)
1 parent 6c41b86 commit b1698da

File tree

2 files changed

+10
-10
lines changed

2 files changed

+10
-10
lines changed

spring-cloud-aws-sqs/src/main/java/io/awspring/cloud/sqs/listener/BatchAwareBackPressureHandler.java

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -44,6 +44,16 @@ default void releaseBatch() {
4444
release(getBatchSize(), ReleaseReason.NONE_FETCHED);
4545
}
4646

47+
@Override
48+
default void release(int amount, ReleaseReason reason) {
49+
if (amount == getBatchSize() && reason == ReleaseReason.NONE_FETCHED) {
50+
releaseBatch();
51+
}
52+
else {
53+
release(amount);
54+
}
55+
}
56+
4757
/**
4858
* Return the configured batch size for this handler.
4959
* @return the batch size.

spring-cloud-aws-sqs/src/main/java/io/awspring/cloud/sqs/listener/SemaphoreBackPressureHandler.java

Lines changed: 0 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -180,16 +180,6 @@ public void release(int amount) {
180180
this.semaphore.availablePermits());
181181
}
182182

183-
@Override
184-
public void release(int amount, ReleaseReason reason) {
185-
if (amount == this.batchSize && reason == ReleaseReason.NONE_FETCHED) {
186-
releaseBatch();
187-
}
188-
else {
189-
release(amount);
190-
}
191-
}
192-
193183
private int getPermitsToRelease(int amount) {
194184
return this.hasAcquiredFullPermits.compareAndSet(true, false)
195185
// The first process that gets here should release all permits except for inflight messages

0 commit comments

Comments
 (0)