Skip to content

Commit 466ef34

Browse files
committed
Allow jobs to not specify proc-level info
There is no requirement that a job specify proc-level info just in order to run. So don't error out if none is available Signed-off-by: Ralph Castain <[email protected]> (cherry picked from commit 4aca3ae)
1 parent 48feba8 commit 466ef34

File tree

1 file changed

+9
-10
lines changed

1 file changed

+9
-10
lines changed

src/mca/gds/hash/gds_hash.c

Lines changed: 9 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -644,25 +644,24 @@ static pmix_status_t register_info(pmix_peer_t *peer,
644644
for (rank=0; rank < ns->nprocs; rank++) {
645645
val = NULL;
646646
rc = pmix_hash_fetch(ht, rank, NULL, &val);
647-
if (PMIX_SUCCESS != rc) {
647+
if (PMIX_SUCCESS != rc && PMIX_ERR_PROC_ENTRY_NOT_FOUND != rc) {
648648
PMIX_ERROR_LOG(rc);
649649
if (NULL != val) {
650650
PMIX_VALUE_RELEASE(val);
651651
}
652652
return rc;
653653
}
654-
if (NULL == val) {
655-
return PMIX_ERR_NOT_FOUND;
656-
}
657654
PMIX_CONSTRUCT(&buf, pmix_buffer_t);
658655
PMIX_BFROPS_PACK(rc, peer, &buf, &rank, 1, PMIX_PROC_RANK);
659656

660-
info = (pmix_info_t*)val->data.darray->array;
661-
ninfo = val->data.darray->size;
662-
for (n=0; n < ninfo; n++) {
663-
kv.key = info[n].key;
664-
kv.value = &info[n].value;
665-
PMIX_BFROPS_PACK(rc, peer, &buf, &kv, 1, PMIX_KVAL);
657+
if (NULL != val) {
658+
info = (pmix_info_t*)val->data.darray->array;
659+
ninfo = val->data.darray->size;
660+
for (n=0; n < ninfo; n++) {
661+
kv.key = info[n].key;
662+
kv.value = &info[n].value;
663+
PMIX_BFROPS_PACK(rc, peer, &buf, &kv, 1, PMIX_KVAL);
664+
}
666665
}
667666
kv.key = PMIX_PROC_BLOB;
668667
kv.value = &blob;

0 commit comments

Comments
 (0)