CMLC#isInExpectedState consistency problem

Fixes: #3063

* change stream to for loops, prevent `ConcurrentModificationException` from simultaneously invoking `KafkaMessageListenerContainer#setStoppedNormally` and `CMLC#isInExpectedState`
* modify property isInExpectedState when invoking stream.allMatch. This was causing list size to change and thus
   throwing `ConcurrentModificationException` by `ArrayList$ArrayListSpliterator#tryAdvance`.

(cherry picked from commit ee835ce)

Author: Wzy19930507

spring-kafka/src/main/java/org/springframework/kafka/listener/ConcurrentMessageListenerContainer.java

@@ -25,7 +25,6 @@
 import java.util.Map;
 import java.util.Objects;
 import java.util.concurrent.atomic.AtomicInteger;
-import java.util.stream.Collectors;
 
 import org.apache.kafka.common.Metric;
 import org.apache.kafka.common.MetricName;
@@ -120,12 +119,12 @@ public void setAlwaysClientIdSuffix(boolean alwaysClientIdSuffix) {
 	public List<KafkaMessageListenerContainer<K, V>> getContainers() {
 		this.lifecycleLock.lock();
 		try {
-			return Collections.unmodifiableList(new ArrayList<>(this.containers));
+			return List.copyOf(this.containers);
 		}
 		finally {
 			this.lifecycleLock.unlock();
 		}
-}
+	}
 
 	@Override
 	public MessageListenerContainer getContainerFor(String topic, int partition) {
@@ -156,7 +155,7 @@ public Collection<TopicPartition> getAssignedPartitions() {
 					.map(KafkaMessageListenerContainer::getAssignedPartitions)
 					.filter(Objects::nonNull)
 					.flatMap(Collection::stream)
-					.collect(Collectors.toList());
+					.toList();
 		}
 		finally {
 			this.lifecycleLock.unlock();
@@ -258,7 +257,6 @@ protected void doStart() {
 		}
 	}
 
-	@SuppressWarnings("deprecation")
 	private void configureChildContainer(int index, KafkaMessageListenerContainer<K, V> container) {
 		String beanName = getBeanName();
 		beanName = (beanName == null ? "consumer" : beanName) + "-" + index;
@@ -307,13 +305,17 @@ private KafkaMessageListenerContainer<K, V> constructContainer(ContainerProperti
 		return container;
 	}
 
+	@Nullable
 	private TopicPartitionOffset[] partitionSubset(ContainerProperties containerProperties, int index) {
 		TopicPartitionOffset[] topicPartitions = containerProperties.getTopicPartitions();
+		if (topicPartitions == null) {
+			return null;
+		}
 		if (this.concurrency == 1) {
-			return topicPartitions; // NOSONAR
+			return topicPartitions;
 		}
 		else {
-			int numPartitions = topicPartitions.length; // NOSONAR
+			int numPartitions = topicPartitions.length;
 			if (numPartitions == this.concurrency) {
 				return new TopicPartitionOffset[] { topicPartitions[index] };
 			}
@@ -388,7 +390,7 @@ && getContainerProperties().isRestartAfterAuthExceptions()
 			if (exec == null) {
 				exec = new SimpleAsyncTaskExecutor(getListenerId() + ".authRestart");
 			}
-			exec.execute(() -> start());
+			exec.execute(this::start);
 		}
 	}
 
@@ -476,10 +478,15 @@ public boolean isPartitionPaused(TopicPartition topicPartition) {
 	public boolean isInExpectedState() {
 		this.lifecycleLock.lock();
 		try {
-			return (isRunning() || isStoppedNormally()) && this.containers
-					.stream()
-					.map(container -> container.isInExpectedState())
-					.allMatch(bool -> Boolean.TRUE.equals(bool));
+			boolean isInExpectedState = isRunning() || isStoppedNormally();
+			if (isInExpectedState) {
+				for (KafkaMessageListenerContainer<K, V> container : this.containers) {
+					if (!container.isInExpectedState()) {
+						return false;
+					}
+				}
+			}
+			return isInExpectedState;
 		}
 		finally {
 			this.lifecycleLock.unlock();