UT minor fix barrier logging

arttianezhu · meta-codesync[bot] · commit aa3bd765dfaf · 2025-10-28T17:57:38.000-07:00
Summary: As title, fix the previous problem where the last rank to arrive at the barrier will not print "barrier leave".

Reviewed By: function47

Differential Revision: D85370482

fbshipit-source-id: af2770a45718a37f9c318638b5b7d2ec0932144a
diff --git a/comms/ctran/tests/bootstrap/IntraProcessBootstrap.cc b/comms/ctran/tests/bootstrap/IntraProcessBootstrap.cc
@@ -37,9 +37,9 @@ void IntraProcessBootstrap::barrierNamed(
       }
       std::this_thread::sleep_for(std::chrono::milliseconds(10));
     }
-
-    CLOGF(INFO, "rank [{}/{}] barrier '{}' leave", rank, nRanks, name);
   }
+
+  CLOGF(INFO, "rank [{}/{}] barrier '{}' leave", rank, nRanks, name);
 }
 
 folly::SemiFuture<int>

Original file line number	Diff line number	Diff line change
`@@ -37,9 +37,9 @@ void IntraProcessBootstrap::barrierNamed(`
`37`	`37`	`}`
`38`	`38`	`std::this_thread::sleep_for(std::chrono::milliseconds(10));`
`39`	`39`	`}`
`40`		`-`
`41`		`- CLOGF(INFO, "rank [{}/{}] barrier '{}' leave", rank, nRanks, name);`
`42`	`40`	`}`
	`41`	`+`
	`42`	`+ CLOGF(INFO, "rank [{}/{}] barrier '{}' leave", rank, nRanks, name);`
`43`	`43`	`}`
`44`	`44`
`45`	`45`	`folly::SemiFuture<int>`