We did talk about whether to index embedded documents in the same map or not. I assume the objective of this PR wasn't to change `documentIdentifiers`, so you're just preserving existing behavior here?