In this particular case ... I just realized because this is `AsyncSubject` it doesn't actually emit here, but why not have the value be an `AtomicReference` like it was before so we don't have locking every time?