Break a race between wreckrun's state change and initial jcb notifica…

…tion
flux-framework · May 28, 2015 · 15b6f87 · 15b6f87
1 parent 251ae18
commit 15b6f87
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/src/modules/libjsc/jstatctl.c b/src/modules/libjsc/jstatctl.c
@@ -798,13 +798,13 @@ static int new_job_cb (const char *key, int64_t val, void *arg, int errnum)
     Jadd_int64 (ss, JSC_STATE_PAIR_NSTATE, (int64_t) js);
     json_object_object_add (jcb, JSC_STATE_PAIR, ss);
 
+    if (invoke_cbs (h, nj, jcb, errnum) < 0) {
+        flux_log (h, LOG_ERR, "new_job_cb: failed to invoke callbacks");
+    }
     if (reg_jobstate_hdlr (h, path, (KVSSetStringF *) job_state_cb) == -1) {
         flux_log (h, LOG_ERR, "new_job_cb: reg_jobstate_hdlr: %s", 
             strerror (errno));
     }
-    if (invoke_cbs (h, nj, jcb, errnum) < 0) {
-        flux_log (h, LOG_ERR, "new_job_cb: failed to invoke callbacks");
-    }
 
 done:
     /* always return 0 so that reactor won't return */