Avoid recomputation of holes in TrackSummaryTool

As of !36404 (merged), the TrackSummaryTool no longer ignores tracks with existing track summaries. However, this has introduced a bit of a performance regression where it recomputes hole searches for tracks that already have that infomation attached. This commit adds a constraint to the hole search code such that it only runs if the track does not already have hole search information attached to it. As far as I can tell this solves the performance bug.

This merge request affects 1 package:

• Tracking/TrkTools/TrkTrackSummaryTool

Adding @amorley as watcher

added Tracking master review-pending-level-1 labels

CI Result SUCCESS (hash aa3c4551)

	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 5
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 20662]

mentioned in merge request !36532 (merged)

Warnings are unrelated, approving. L1 reviewer.

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

added bugfix label

This fixes ATLASRECTS-5668.

merged

mentioned in commit ab17d019

added sweep:ignore label