diff --git a/InnerDetector/InDetRecAlgs/InDetPrepRawDataFormation/src/CacheCreator.cxx b/InnerDetector/InDetRecAlgs/InDetPrepRawDataFormation/src/CacheCreator.cxx index 96e139890f52e038fb445308563b7955e552a547..ee0748889c00a8bdd8a412118f11d062729202ca 100644 --- a/InnerDetector/InDetRecAlgs/InDetPrepRawDataFormation/src/CacheCreator.cxx +++ b/InnerDetector/InDetRecAlgs/InDetPrepRawDataFormation/src/CacheCreator.cxx @@ -66,8 +66,12 @@ namespace InDet{ StatusCode CacheCreator::execute (const EventContext& ctx) const { - if(!m_disableWarning && isInsideView(ctx)){ - ATH_MSG_WARNING("CacheCreator is running inside a view, this is probably a misconfiguration"); + if(!m_disableWarning){ + if(isInsideView(ctx)){ + ATH_MSG_ERROR("CacheCreator is running inside a view, this is probably a misconfiguration"); + return StatusCode::FAILURE; + } + m_disableWarning = true; //only check once } if(!m_disableTRT) ATH_CHECK(createContainer(m_rioContainerCacheKey, m_pTRTHelper->straw_layer_hash_max(), ctx)); diff --git a/InnerDetector/InDetRecAlgs/InDetPrepRawDataFormation/src/CacheCreator.h b/InnerDetector/InDetRecAlgs/InDetPrepRawDataFormation/src/CacheCreator.h index d18484bcfbebf31c9c3bd8fcd6ffae48119bd8d6..16cc736a0568c03b297c0a81fe85b9825156e2b6 100644 --- a/InnerDetector/InDetRecAlgs/InDetPrepRawDataFormation/src/CacheCreator.h +++ b/InnerDetector/InDetRecAlgs/InDetPrepRawDataFormation/src/CacheCreator.h @@ -51,7 +51,7 @@ namespace InDet{ SG::WriteHandleKey<SCT_RDO_Cache> m_SCTRDOCacheKey; SG::WriteHandleKey<PixelRDO_Cache> m_PixRDOCacheKey; bool m_disableTRT; - bool m_disableWarning; + mutable bool m_disableWarning; //Temporary workarounds for problem in scheduler - remove later SG::ReadCondHandleKey<PixelCalib::PixelOfflineCalibData> m_condKey5{ this, "PixelOfflineCalibData", "PixelOfflineCalibData", "" }; SG::ReadCondHandleKey<DetectorSpecialPixelMap> m_condKey12{ this, "SpecialPixelMap", "SpecialPixelMap", "" }; diff --git a/MuonSpectrometer/MuonCnv/MuonByteStream/src/MuonCacheCreator.cxx b/MuonSpectrometer/MuonCnv/MuonByteStream/src/MuonCacheCreator.cxx index f13d808a4c4420d8c2de5a7db80cac6a46d5d7a8..b55dc1ff486748e11997163b8ea21ea643266cc4 100644 --- a/MuonSpectrometer/MuonCnv/MuonByteStream/src/MuonCacheCreator.cxx +++ b/MuonSpectrometer/MuonCnv/MuonByteStream/src/MuonCacheCreator.cxx @@ -37,8 +37,12 @@ bool MuonCacheCreator::isInsideView(const EventContext& context) const StatusCode MuonCacheCreator::execute (const EventContext& ctx) const { - if(!m_disableWarning && isInsideView(ctx)){ - ATH_MSG_WARNING("CacheCreator is running inside a view, this is probably a misconfiguration"); + if(!m_disableWarning){ + if(isInsideView(ctx)){ + ATH_MSG_ERROR("CacheCreator is running inside a view, this is probably a misconfiguration"); + return StatusCode::FAILURE; + } + m_disableWarning = true; //only check once } // Create the MDT cache container auto maxHashMDTs = m_mdtIdHelper->stationNameIndex("BME") != -1 ? diff --git a/MuonSpectrometer/MuonCnv/MuonByteStream/src/MuonCacheCreator.h b/MuonSpectrometer/MuonCnv/MuonByteStream/src/MuonCacheCreator.h index c3908ac00b8b3e67ad2b486260c87fc101d1d252..9a78b8520d0eb134dca194fae98136c1ed60f420 100644 --- a/MuonSpectrometer/MuonCnv/MuonByteStream/src/MuonCacheCreator.h +++ b/MuonSpectrometer/MuonCnv/MuonByteStream/src/MuonCacheCreator.h @@ -34,7 +34,7 @@ protected: /// ID helpers const MdtIdHelper* m_mdtIdHelper = 0; - bool m_disableWarning = false; + mutable bool m_disableWarning = false; bool isInsideView(const EventContext&) const; };//class MuonCacheCreator