Merge pull request #814 from xiaoerlyl/master

Add callback to defferdResult before putting to the map

apollo-configservice/src/main/java/com/ctrip/framework/apollo/configservice/controller/NotificationControllerV2.java

@@ -150,11 +150,6 @@ public class NotificationControllerV2 implements ReleaseMessageListener {
     if (!CollectionUtils.isEmpty(newNotifications)) {
       deferredResultWrapper.setResult(newNotifications);
     } else {
-      //register all keys
-      for (String key : watchedKeys) {
-        this.deferredResults.put(key, deferredResultWrapper);
-      }
-
       deferredResultWrapper
           .onTimeout(() -> logWatchedKeys(watchedKeys, "Apollo.LongPoll.TimeOutKeys"));
 
@@ -166,6 +161,11 @@ public class NotificationControllerV2 implements ReleaseMessageListener {
         logWatchedKeys(watchedKeys, "Apollo.LongPoll.CompletedKeys");
       });
 
+      //register all keys
+      for (String key : watchedKeys) {
+        this.deferredResults.put(key, deferredResultWrapper);
+      }
+
       logWatchedKeys(watchedKeys, "Apollo.LongPoll.RegisteredKeys");
       logger.debug("Listening {} from appId: {}, cluster: {}, namespace: {}, datacenter: {}",
           watchedKeys, appId, cluster, namespaces, dataCenter);