GH-2504: Clear Deffered Channel Close Executor

Addendum to #2504 - when the connection is reset, the `channelsExecutor`
is shut down. It needs to be reset so that a new one is created, if necessary.

Also fix synchronization.

**cherry-pick to 2.4.x**

(cherry picked from commit 5dd01b7)

Author: garyrussell

spring-rabbit/src/main/java/org/springframework/amqp/rabbit/connection/CachingConnectionFactory.java

@@ -860,18 +860,24 @@ public final void destroy() {
 		resetConnection();
 		if (getContextStopped()) {
 			this.stopped = true;
-			if (this.channelsExecutor != null) {
-				try {
-					if (!this.inFlightAsyncCloses.await(CHANNEL_EXEC_SHUTDOWN_TIMEOUT, TimeUnit.SECONDS)) {
-						this.logger.warn("Async closes are still in-flight: " + this.inFlightAsyncCloses.getCount());
+			synchronized (this.connectionMonitor) {
+				if (this.channelsExecutor != null) {
+					try {
+						if (!this.inFlightAsyncCloses.await(CHANNEL_EXEC_SHUTDOWN_TIMEOUT, TimeUnit.SECONDS)) {
+							this.logger
+									.warn("Async closes are still in-flight: " + this.inFlightAsyncCloses.getCount());
+						}
+						this.channelsExecutor.shutdown();
+						if (!this.channelsExecutor.awaitTermination(CHANNEL_EXEC_SHUTDOWN_TIMEOUT, TimeUnit.SECONDS)) {
+							this.logger.warn("Channel executor failed to shut down");
+						}
 					}
-					this.channelsExecutor.shutdown();
-					if (!this.channelsExecutor.awaitTermination(CHANNEL_EXEC_SHUTDOWN_TIMEOUT, TimeUnit.SECONDS)) {
-						this.logger.warn("Channel executor failed to shut down");
+					catch (@SuppressWarnings(UNUSED) InterruptedException e) {
+						Thread.currentThread().interrupt();
+					}
+					finally {
+						this.channelsExecutor = null;
 					}
-				}
-				catch (@SuppressWarnings(UNUSED) InterruptedException e) {
-					Thread.currentThread().interrupt();
 				}
 			}
 		}