In the OperationContextImpl destructor we create a new locker then immediately destroy it. If the point of caching is that construction/destruction of LockState is too expensive, then we are are still paying the cost and getting nothing out of it. We should consider removing the cache rather than fixing it.