AthenaKernel+AthenaPoolCnvSvc: Add support for running multiple new SharedWriter instances in parallel
As discussed in ATEAM-739, currently one needs to explicitly set the port number in order to run multiple instances of the new SharedWriter
in parallel. This MR changes this behavior. Now we delegate the port selection to ROOT to avoid collisions.
I'll leave this in Draft
mode until @gemmeren gives the go ahead. I locally tested the changes by running four AthenaMP instance in parallel on the same machine and things seem to work as expected. Nonetheless, an independent check/confirmation is, of course, most welcome.
Merge request reports
Activity
- Resolved by Alaettin Serhan Mete
Thanks @amete , this looks good (and I seem to be able to mark it as ready). Cheers, Peter
added Core Database master review-pending-level-1 labels
removed review-pending-level-1 label
CI Result FAILURE (hash 3bffdfee) Athena AthSimulation AthGeneration AnalysisBase AthAnalysis DetCommon externals cmake make required tests optional tests Full details available on this CI monitor view
Athena: number of compilation errors 1, warnings 0
AthSimulation: number of compilation errors 0, warnings 0
AthGeneration: number of compilation errors 0, warnings 0
AnalysisBase: number of compilation errors 0, warnings 0
AthAnalysis: number of compilation errors 0, warnings 0
DetCommon: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 32711] added review-pending-level-1 label
CI Result SUCCESS (hash 3bffdfee) Athena AthSimulation AthGeneration AnalysisBase AthAnalysis DetCommon externals cmake make required tests optional tests Full details available on this CI monitor view
Athena: number of compilation errors 0, warnings 0
AthSimulation: number of compilation errors 0, warnings 0
AthGeneration: number of compilation errors 0, warnings 0
AnalysisBase: number of compilation errors 0, warnings 0
AthAnalysis: number of compilation errors 0, warnings 0
DetCommon: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 32746] added review-approved label and removed review-pending-level-1 label
mentioned in commit 4d2e0877
added sweep:ignore label
mentioned in merge request !43291 (merged)