INT-2436: JMS channel: bean name for container (#3565)

JIRA: https://jira.spring.io/browse/INT-2436

The `JmsChannelFactoryBean` creates a `ListenerContainer` internally
without any `beanName` propagation.
When we rely on a default internal `Executor`, it is created with a
default thread name prefix for all the JMS channel instances.
It cause a confusion in logs

* Set `beanName` for the internal `ListenerContainer` to `this.beanName + ".container"`
making its connection with a channel it is associated with and unique thread name prefix
* Comment out `allWarningsAsErrors = true` in `build.gradle` for deprecated Kotlin
language version `1.3`.
Otherwise IDEA doesn't want to build project

Author: artembilan

build.gradle

@@ -206,7 +206,7 @@ configure(javaProjects) { subproject ->
 			languageVersion = '1.3'
 			jvmTarget = '1.8'
 			freeCompilerArgs = ['-Xjsr305=strict']
-			allWarningsAsErrors = true
+//			allWarningsAsErrors = true TODO bring it back after upgrading to Kotlin 1.5 language version
 		}
 	}
 	compileTestKotlin {

spring-integration-jms/src/main/java/org/springframework/integration/jms/config/JmsChannelFactoryBean.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2002-2020 the original author or authors.
+ * Copyright 2002-2021 the original author or authors.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -66,8 +66,6 @@ public class JmsChannelFactoryBean extends AbstractFactoryBean<AbstractJmsChanne
 
 	private final JmsTemplate jmsTemplate = new DynamicJmsTemplate();
 
-	private AbstractMessageListenerContainer listenerContainer;
-
 	private Class<? extends AbstractMessageListenerContainer> containerType;
 
 	private boolean acceptMessagesWhileStopping;
@@ -376,9 +374,9 @@ public Class<?> getObjectType() {
 	protected AbstractJmsChannel createInstance() {
 		this.initializeJmsTemplate();
 		if (this.messageDriven) {
-			this.listenerContainer = createContainer();
+			AbstractMessageListenerContainer listenerContainer = createContainer();
 			SubscribableJmsChannel subscribableJmsChannel =
-					new SubscribableJmsChannel(this.listenerContainer, this.jmsTemplate);
+					new SubscribableJmsChannel(listenerContainer, this.jmsTemplate);
 			subscribableJmsChannel.setMaxSubscribers(this.maxSubscribers);
 			this.channel = subscribableJmsChannel;
 		}
@@ -414,6 +412,7 @@ private AbstractMessageListenerContainer createContainer() {
 			this.containerType = DefaultMessageListenerContainer.class;
 		}
 		AbstractMessageListenerContainer container = BeanUtils.instantiateClass(this.containerType);
+		container.setBeanName(this.beanName + ".container");
 		container.setAcceptMessagesWhileStopping(this.acceptMessagesWhileStopping);
 		container.setAutoStartup(this.autoStartup);
 		container.setClientId(this.clientId);
@@ -436,8 +435,6 @@ private AbstractMessageListenerContainer createContainer() {
 		container.setSubscriptionDurable(this.subscriptionDurable);
 		container.setSubscriptionShared(this.subscriptionShared);
 
-
-
 		if (container instanceof DefaultMessageListenerContainer) {
 			DefaultMessageListenerContainer dmlc = (DefaultMessageListenerContainer) container;
 			JavaUtils.INSTANCE

spring-integration-jms/src/test/java/org/springframework/integration/jms/config/JmsChannelParserTests.java

@@ -150,6 +150,8 @@ public void queueNameChannel() {
 		assertThat(TestUtils.getPropertyValue(
 				TestUtils.getPropertyValue(channel, "dispatcher"), "maxSubscribers", Integer.class).intValue())
 				.isEqualTo(1);
+		assertThat(TestUtils.getPropertyValue(container, "taskExecutor.threadNamePrefix"))
+				.isEqualTo("queueNameChannel.container-");
 	}
 
 	@Test