Skip to content
Snippets Groups Projects

AthenaKernel+AthenaPoolCnvSvc: Add support for running multiple new SharedWriter instances in parallel

Merged Alaettin Serhan Mete requested to merge amete/athena:master-ATEAM-739 into master
All threads resolved!

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.

Edited by Alaettin Serhan Mete

Merge request reports

Pipeline #2552127 passed

Pipeline passed for 3bffdfee on amete:master-ATEAM-739

Merged by Adam Edward BartonAdam Edward Barton 3 years ago (Apr 30, 2021 1:19pm UTC)

Merge details

  • Changes merged into with 4d2e0877.
  • Deleted the source branch.

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading