More PR feedback.

Author: kinyoklion

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

@@ -144,7 +144,7 @@ public void onResponse(@Nonnull Call call, @Nonnull Response response) {
                         FDv2PayloadResponse pollingResponse = new FDv2PayloadResponse(events, response.headers());
                         future.complete(pollingResponse);
 
-                    } catch (IOException | SerializationException e) {
+                    } catch (Exception e) {
                         future.completeExceptionally(e);
                     } finally {
                         response.close();

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

@@ -146,9 +146,12 @@ private void runSynchronizers() {
                                     // TODO: We may need logging or to do a little more.
                                     return;
                                 case TERMINAL_ERROR:
-                                case GOODBYE:
+                                    availableSynchronizer.block();
                                     running = false;
                                     break;
+                                case GOODBYE:
+                                    // We let the synchronizer handle this internally.
+                                    break;
                             }
                             break;
                     }

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

@@ -38,7 +38,6 @@ public PollingSynchronizerImpl(
     private void doPoll() {
         try {
             FDv2SourceResult res = poll(selectorSource.getSelector(), false).get();
-            boolean shouldSignalShutDown = false;
             switch(res.getResultType()) {
                 case CHANGE_SET:
                     break;