TRT_RawDataByteStreamCnv: threading fixes

Rework to avoid calling naughtyRetrieve.

Remove ATLAS_NOT_THREAD_SAFE annotations --- just need to avoid a (needless!) AttributeList copy.

Make sure to protect TRTRawDataProviderTool::m_cache with a lock. Not safe to omit it.

Note that the original code was sometimes unpacking the same channel multiple times into the same collection. This behavior is fixed by the change, but it results in RootComp differences for a couple TriggerTest ART tests.

This merge request affects 1 package:

• InnerDetector/InDetEventCnv/TRT_RawDataByteStreamCnv

Adding @cgrefe ,@calfayan ,@sroe ,@goetz as watchers

added InnerDetector master review-pending-level-1 labels

CI Result FAILURE (hash 6fcf9d0a)

	Athena	AthSimulation	AthGeneration	AnalysisBase
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
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 20339]

Failing tests are showing up in several MR and are unrelated, approving.

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

merged

mentioned in commit 14c2d0af

added sweep:ignore label