Sync MPI processes before filesystem ops

Sbozzolo · Sbozzolo · commit d889c5cc60c3 · 2024-12-05T13:15:22.000-08:00
`generate_output_path` operates on the filesystem, so it is wise to make
sure that all the processes are in sync
diff --git a/src/OutputPathGenerator.jl b/src/OutputPathGenerator.jl
@@ -137,6 +137,8 @@ function generate_output_path(
     style::OutputPathGeneratorStyle = ActiveLinkStyle(),
 )
     output_path == "" && error("output_path cannot be empty")
+    # Let's make sure we are synced before we do any filesystem operation
+    ClimaComms.barrier(context)
     return generate_output_path(style, output_path; context)
 end
 

Original file line number	Diff line number	Diff line change
`@@ -137,6 +137,8 @@ function generate_output_path(`
`137`	`137`	`style::OutputPathGeneratorStyle = ActiveLinkStyle(),`
`138`	`138`	`)`
`139`	`139`	`output_path == "" && error("output_path cannot be empty")`
	`140`	`+ # Let's make sure we are synced before we do any filesystem operation`
	`141`	`+ ClimaComms.barrier(context)`
`140`	`142`	`return generate_output_path(style, output_path; context)`
`141`	`143`	`end`
`142`	`144`