We seem to do eager reads for all keys in storage server, even when not needed

We do eager reads for all keys in relevant mutations (such a atomics, clearRange etc). We do it irrespective of whether we need it or not. In some cases, we may just need to read the versioned map for example.

See this code snippet from update()

for(auto& c : fii.changes)
eager.addMutations(c.mutations);

wait( doEagerReads( data, &eager ) );
if (data->shardChangeCounter == changeCounter) break;

Evaluate whether this should be optimized or there is a reason for it.