Skip to content

Commit

Permalink
Merge pull request #24 from DilumAluthge-forks/dpa/rethrow-error
Browse files Browse the repository at this point in the history
If an error is encountered, print the log message with `@error` (instead of `println`)
  • Loading branch information
kleinhenz authored Jan 16, 2025
2 parents 2b7be50 + 3b7bfe1 commit 3f7dd5e
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions src/slurmmanager.jl
Original file line number Diff line number Diff line change
Expand Up @@ -151,9 +151,9 @@ function launch(manager::SlurmManager, params::Dict, instances_arr::Array, c::Co
sleep(manager.srun_post_exit_sleep)
end

catch e
println("Error launching Slurm job:")
rethrow(e)
catch ex
@error "Error launching Slurm job" exception=ex
rethrow(ex)
end
end

Expand Down

0 comments on commit 3f7dd5e

Please sign in to comment.