ReadConcern is rarely nil and is instead usually a zero valued struct when created by a default client. The old code for addReadConcern used to check if the encoded result had a length of non-zero before adding it to the document. This check is gone and an empty document is encoded. This same check could be added back or a different check can be done to optimize performance where we don't need to marshal.