Summary

TrafficRecorder::observe() builds a BSON representation of the transport session (ts.toBSON().toString()) on every operation before checking whether recording is enabled. When traffic recording is disabled (the default), this serialization work is wasted.

Profiling shows 1.3% of CPU spent in this path on a Graviton instance running YCSB 100 read.

Proposed Fix

Add an early return check of _shouldRecord in observe() before the expensive serialization:

See this patch for change. 

Impac

This multipatch showed a 2.89% improvment in YCSB 100 read with the proposed change.