diff options
author | Tor Egge <Tor.Egge@oath.com> | 2018-05-29 12:43:07 +0000 |
---|---|---|
committer | Tor Egge <Tor.Egge@oath.com> | 2018-05-29 12:43:07 +0000 |
commit | 6fef8fd0231ae40586ed8d20110d91f589647227 (patch) | |
tree | 2d73ff20bd4f60191966b5b5babfd85bd9c029d1 /searchcore | |
parent | 302146b7ca558f99b5d990abc75c2d0c2ebeeef3 (diff) |
Add comment explaining why identity check is sufficient.
Diffstat (limited to 'searchcore')
-rw-r--r-- | searchcore/src/vespa/searchcore/proton/server/feedhandler.cpp | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/searchcore/src/vespa/searchcore/proton/server/feedhandler.cpp b/searchcore/src/vespa/searchcore/proton/server/feedhandler.cpp index 897e58139c1..90b9bbc7f34 100644 --- a/searchcore/src/vespa/searchcore/proton/server/feedhandler.cpp +++ b/searchcore/src/vespa/searchcore/proton/server/feedhandler.cpp @@ -96,6 +96,11 @@ void FeedHandler::performPut(FeedToken token, PutOperation &op) { } return; } + /* + * Check if document type repos are equal. DocumentTypeRepoFactory::make + * returns the same document type repo if document type configs are equal, + * thus we can just perform a cheaper identity check here. + */ if (_repo != op.getDocument()->getRepo()) { op.deserializeDocument(*_repo); } |