Fix spurious / irrelevant data race in DataObjectHandle and remove unused state

Fix a data race flagged by helgrind which swamps any helgrind log -- the actual race is itself is harmless: it could lead to rare spurious double work, but without observable effect other than spending some cycles. The fix does imply an additional atomic bool, which may have a bigger performance impact, but is presumably small enough to also not have an observable difference...

added code cleanup thread-safety labels

approved this merge request

changed the description

added 1 commit

• def3cc10 - Fix spurious / irrelevant data race in DataObjectHandle and remove unused state

Compare with previous version

added 1 commit

• a8b9d6d3 - Fix spurious / irrelevant data race in DataObjectHandle and remove unused state

Compare with previous version

added 1 commit

• 5d21d5d9 - Fix spurious / irrelevant data race in DataObjectHandle and remove unused state

Compare with previous version

added 1 commit

• d60ab001 - Fix spurious / irrelevant data race in DataObjectHandle and remove unused state

Compare with previous version

approved this merge request

added 1 commit

• 34ea0c18 - Fix spurious / irrelevant data race in DataObjectHandle and remove unused state

Compare with previous version

added 1 commit

• 81b3311b - Fix spurious / irrelevant data race in DataObjectHandle and remove unused state

Compare with previous version

approved this merge request

changed milestone to %v36r5

added lhcb-gaudi-head label

assigned to @clemenci

• [2022-03-24 00:04] Validation started with lhcb-gaudi-head#3196
• [2022-03-24 00:34] Validation started with lhcb-run2-gaudi-head#295
• [2022-03-25 00:06] Validation started with lhcb-gaudi-head#3197
• [2022-03-25 00:39] Validation started with lhcb-run2-gaudi-head#296

mentioned in commit aab42633

merged