Since we are merging a "quick" version in order to unblock other parts of the project, we should come back at the end to optimize them. They currently rely on multiple BSONObjBuilders which hold intermediate state about the diff. This should be avoided.
We should also add more test cases for situations where a DocumentDiffReader is given an invalid/malformed diff.