Use ActiveStoreSvc to pick event store Fix ATLASSIM-4604

This merge request affects 1 package:

Event/ByteStreamCnvSvc

Adding @tadej as watcher

added EDM master review-pending-level-1 labels

Sorry for naive question, but what sets the active store?

TBH I don't know or understand how that service works.

For debugging I had the eventStore function print out the name of the event store it uses. It does come up with what ByteStreamEventSelectorInputSvc is set to ... but not the _Impl then BCMOverlay's handle is looking for.

It looks like it is set in StoreGateSvc::loadEventProxies(). That is called in the Athena(Hive)EventLoopManager when starting an event or handling an incident.

I wonder if this is needed at all if we could just manually set the store to the event selector. I would avoid making the code more complicated just for something that's moving away anyways soon.

Let me try to change configuration tomorrow now that we know what the issue is. Sorry if you needed much work to implement this.

CI Result SUCCESS (hash 54c6f241)

	Athena	AthSimulation	AnalysisBase	AthGeneration
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
AnalysisBase: number of compilation errors 0, warnings 0
AthGeneration: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 15371]

Could you please resolve the discussions when you are happy with the outcome. Changing label back to 'user-action-required' until the final discussion is resolved.

Thanks. Xiaozhong (L1)