Currently we take a global lock when on operator open. Is this the right approach?

Lock::GlobalLock lock(opCtx, MODE_IS);

Without this locking we can fail parallel plans with indexes by asserting here

boost::optional<KeyStringEntry> seekForKeyString(
const KeyString::Value& keyStringValue) override {
dassert(_opCtx>lockState()>isReadLocked());