ulfm/comm_mark_failed: remove debug that causes all procs to be

populated, this is too agressive and can mask bugs in debug mode

Signed-off-by: Aurelien Bouteiller <bouteill@icl.utk.edu>

Author: abouteiller

ompi/communicator/ft/comm_ft.c

@@ -334,17 +334,6 @@ bool ompi_comm_is_proc_active(ompi_communicator_t *comm, int peer_id, bool remot
     /* If the proc is not known yet (get_proc_ptr returns NULL for a valid
      * peer_id), then we assume that the proc is alive. When it is dead, the
      * proc will exist. */
-#if OPAL_ENABLE_DEBUG
-    if(NULL == ompi_proc) {
-        /* this debug has side effects on behavior/performance: it loads up the
-         * proc for every query and may end-up being equivalent to the 'preconnect
-         * all' option in the worse case. */
-        ompi_proc = ompi_group_get_proc_ptr((remote ? comm->c_remote_group : comm->c_local_group),
-                                             peer_id, true);
-        assert(NULL != ompi_proc);
-        assert(ompi_proc_is_active(ompi_proc));
-    }
-#endif
     return (NULL == ompi_proc) ? true : ompi_proc_is_active(ompi_proc);
 }