Fix Sonar issues

spring-projects · Dec 5, 2019 · fde3b20 · fde3b20
1 parent 459b34c
commit fde3b20
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 5 deletions.
diff --git a/spring-kafka-test/src/main/java/org/springframework/kafka/test/utils/ContainerTestUtils.java b/spring-kafka-test/src/main/java/org/springframework/kafka/test/utils/ContainerTestUtils.java
@@ -46,7 +46,7 @@ private ContainerTestUtils() {
 	 * @throws ContainerTestUtilsException if the call to the container's
 	 * getAssignedPartitions() method fails.
 	 */
-	public static void waitForAssignment(Object container, int partitions) {
+	public static void waitForAssignment(Object container, int partitions) { // NOSONAR complexity
 		if (container.getClass().getSimpleName().contains("KafkaMessageListenerContainer")) {
 			waitForSingleContainerAssignment(container, partitions);
 			return;

diff --git a/...kafka/src/main/java/org/springframework/kafka/listener/KafkaMessageListenerContainer.java b/...kafka/src/main/java/org/springframework/kafka/listener/KafkaMessageListenerContainer.java
@@ -1558,10 +1558,8 @@ private ConsumerRecord<K, V> checkEarlyIntercept(ConsumerRecord<K, V> nextArg) {
 			ConsumerRecord<K, V> next = nextArg;
 			if (this.earlyRecordInterceptor != null) {
 				next = this.earlyRecordInterceptor.intercept(next);
-				if (next == null) {
-					if (this.logger.isDebugEnabled()) {
-						this.logger.debug("RecordInterceptor returned null, skipping: " + next);
-					}
+				if (next == null && this.logger.isDebugEnabled()) {
+					this.logger.debug("RecordInterceptor returned null, skipping: " + nextArg);
 				}
 			}
 			return next;