Skip to content

Commit 788e31e

Browse files
committed
checkstyle
1 parent 68ff1f5 commit 788e31e

File tree

1 file changed

+8
-2
lines changed

1 file changed

+8
-2
lines changed

src/main/java/com/arpnetworking/clusteraggregator/configuration/ClusterAggregatorConfiguration.java

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -603,9 +603,15 @@ public Builder setCalculateClusterAggregations(final Boolean value) {
603603
return this;
604604
}
605605

606+
/**
607+
* Validate that the aggregator liveliness timeout is less than the reaggregation timeout.
608+
*
609+
* @param aggregatorLivelinessTimeout the configured liveliness timeout
610+
* @return true if the given value is valid
611+
*/
606612
@SuppressFBWarnings(value = "UPM_UNCALLED_PRIVATE_METHOD", justification = "invoked reflectively by @ValidateWithMethod")
607613
public boolean validateAggregatorLivelinessTimeout(final Duration aggregatorLivelinessTimeout) {
608-
return aggregatorLivelinessTimeout.compareTo(_reaggregationTimeout) < 0;
614+
return aggregatorLivelinessTimeout.compareTo(_reaggregationTimeout) > 0;
609615
}
610616

611617
@NotNull
@@ -655,7 +661,7 @@ public boolean validateAggregatorLivelinessTimeout(final Duration aggregatorLive
655661
@NotNull
656662
private Duration _reaggregationTimeout = Duration.ofMinutes(1);
657663
@NotNull
658-
@ValidateWithMethod(methodName = "validateAggregatorLivelinessTimeout", parameterType=Duration.class)
664+
@ValidateWithMethod(methodName = "validateAggregatorLivelinessTimeout", parameterType = Duration.class)
659665
private Duration _aggregatorLivelinessTimeout = _reaggregationTimeout.multipliedBy(2);
660666
@NotNull
661667
private File _hostPipelineConfiguration;

0 commit comments

Comments
 (0)