Skip to content

[ATR-21910][ATR-21910] Trig Cost online concurrency fixes

Tim Martin requested to merge tamartin/athena:trigCostConcurrency into master

Resolve two issues seen at P1 in MT mode.

  1. Multiple slots could access the thread-hash -> thread-counter map concurrently. The mutex protection was only per-slot. Resolved by adding a small global lock to this map lookup.
  2. We encountered hash collisions online. As we only collect monitoring data here, this is not too critical and the exception can be caught and discarded with only a DEBUG message. A unit test was added around this.

Merge request reports