Adding PersSvcPerInputType property to allow using multiple persistency...

Adding PersSvcPerInputType property to allow using multiple persistency services, one per input type. This property is defaulted to false -> no change. But can be switched on for MT jobs that are input limited and may help with https://its.cern.ch/jira/browse/ATEAM-740

This merge request affects 1 package:

• Database/AthenaPOOL/AthenaPoolCnvSvc

This merge request affects 2 files:

• Database/AthenaPOOL/AthenaPoolCnvSvc/src/AthenaPoolCnvSvc.cxx
• Database/AthenaPOOL/AthenaPoolCnvSvc/src/AthenaPoolCnvSvc.h

Adding @mnowak ,@ssnyder as watchers

added Database master review-pending-level-1 labels

Hi @tadej

I created this MR with somewhat cleaned up changes. By default a single persistency svc is used (no change), but for pile up you can switch on using multiple persistency services, one per input type:

SvcMgr.AthenaPoolCnvSvc.PersSvcPerInputType = True

Hi @tadej

I created this MR with somewhat cleaned up changes. By default a single persistency svc is used (no change), but for pile up you can switch on using multiple persistency services, one per input type:

SvcMgr.AthenaPoolCnvSvc.PersSvcPerInputType = True

Hi @gemmeren, so this means one service is used for hard-scatter HITS and one for the background RDO?

Hi Tadej,

this goes a step further, using different services per data type even within the same input file. The multiple/parallel input file use-case is rather special and this way we should gain concurrency even with a single input stream.

however, if this doesn't work out, we could add separate service per input stream.

CI Result SUCCESS (hash 56b9b240)

	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 33022]

added review-approved label and removed review-pending-level-1 label

mentioned in commit f3639e2c

merged

added sweep:ignore label