PR feedback.

Author: kinyoklion

lib/sdk/server/src/main/java/com/launchdarkly/sdk/server/FDv2DataSource.java

@@ -44,13 +44,7 @@ public enum State {
             /**
              * This synchronizer is no longer available to use.
              */
-            Blocked,
-
-            /**
-             * This synchronizer is recovering from a previous failure and will be available to use
-             * after a delay.
-             */
-            Recovering
+            Blocked
         }
 
         private final SynchronizerFactory factory;
@@ -70,11 +64,6 @@ public void block() {
             state = State.Blocked;
         }
 
-        public void setRecovering(Duration delay) {
-            state = State.Recovering;
-            // TODO: Determine how/when to recover.
-        }
-
         public Synchronizer build() {
             return factory.build();
         }
@@ -234,13 +223,6 @@ public boolean isInitialized() {
 
     @Override
     public void close() throws IOException {
-        // If this is already set, then this has no impact.
-        startFuture.complete(false);
-        synchronized (synchronizers) {
-            for (SynchronizerFactoryWithState synchronizer : synchronizers) {
-                synchronizer.block();
-            }
-        }
         // If there is an active source, we will shut it down, and that will result in the loop handling that source
         // exiting.
         // If we do not have an active source, then the loop will check isShutdown when attempting to set one. When
@@ -251,5 +233,8 @@ public void close() throws IOException {
                 activeSource.shutdown();
             }
         }
+
+        // If this is already set, then this has no impact.
+        startFuture.complete(false);
     }
 }