Skip to content

Commit 70a7274

Browse files
committed
Fix Sonar Issues
1 parent 12c3540 commit 70a7274

File tree

1 file changed

+13
-5
lines changed

1 file changed

+13
-5
lines changed

Diff for: spring-kafka/src/main/java/org/springframework/kafka/listener/PartitionPausingBackOffManagerFactory.java

+13-5
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,7 @@ public class PartitionPausingBackOffManagerFactory extends AbstractKafkaBackOffM
5151
*/
5252
public PartitionPausingBackOffManagerFactory(KafkaConsumerTimingAdjuster timingAdjustmentManager) {
5353
this.clock = getDefaultClock();
54-
setTimingAdjustmentManager(timingAdjustmentManager);
54+
doSetTimingAdjustmentManager(timingAdjustmentManager);
5555
}
5656

5757
/**
@@ -62,7 +62,7 @@ public PartitionPausingBackOffManagerFactory(KafkaConsumerTimingAdjuster timingA
6262
*/
6363
public PartitionPausingBackOffManagerFactory(TaskExecutor timingAdjustmentManagerTaskExecutor) {
6464
this.clock = getDefaultClock();
65-
setTaskExecutor(timingAdjustmentManagerTaskExecutor);
65+
doSetTaskExecutor(timingAdjustmentManagerTaskExecutor);
6666
}
6767

6868
/**
@@ -73,7 +73,7 @@ public PartitionPausingBackOffManagerFactory(TaskExecutor timingAdjustmentManage
7373
*/
7474
public PartitionPausingBackOffManagerFactory(boolean timingAdjustmentEnabled) {
7575
this.clock = getDefaultClock();
76-
setTimingAdjustmentEnabled(timingAdjustmentEnabled);
76+
this.timingAdjustmentEnabled = timingAdjustmentEnabled;
7777
}
7878

7979
/**
@@ -118,7 +118,11 @@ public void setTimingAdjustmentEnabled(boolean timingAdjustmentEnabled) {
118118
*
119119
* @param timingAdjustmentManager the adjustmentManager to be used.
120120
*/
121-
public final void setTimingAdjustmentManager(KafkaConsumerTimingAdjuster timingAdjustmentManager) {
121+
public void setTimingAdjustmentManager(KafkaConsumerTimingAdjuster timingAdjustmentManager) {
122+
doSetTimingAdjustmentManager(timingAdjustmentManager);
123+
}
124+
125+
private void doSetTimingAdjustmentManager(KafkaConsumerTimingAdjuster timingAdjustmentManager) {
122126
Assert.isTrue(this.timingAdjustmentEnabled, () -> "TimingAdjustment is disabled for this factory.");
123127
this.timingAdjustmentManager = timingAdjustmentManager;
124128
}
@@ -127,7 +131,11 @@ public final void setTimingAdjustmentManager(KafkaConsumerTimingAdjuster timingA
127131
* Set the {@link TaskExecutor} that will be used in the {@link KafkaConsumerTimingAdjuster}.
128132
* @param taskExecutor the taskExecutor to be used.
129133
*/
130-
public final void setTaskExecutor(TaskExecutor taskExecutor) {
134+
public void setTaskExecutor(TaskExecutor taskExecutor) {
135+
doSetTaskExecutor(taskExecutor);
136+
}
137+
138+
private void doSetTaskExecutor(TaskExecutor taskExecutor) {
131139
Assert.isTrue(this.timingAdjustmentEnabled, () -> "TimingAdjustment is disabled for this factory.");
132140
this.taskExecutor = taskExecutor;
133141
}

0 commit comments

Comments
 (0)