summaryrefslogtreecommitdiffstats
path: root/searchcore
diff options
context:
space:
mode:
authorTor Egge <Tor.Egge@oath.com>2018-05-29 12:43:07 +0000
committerTor Egge <Tor.Egge@oath.com>2018-05-29 12:43:07 +0000
commit6fef8fd0231ae40586ed8d20110d91f589647227 (patch)
tree2d73ff20bd4f60191966b5b5babfd85bd9c029d1 /searchcore
parent302146b7ca558f99b5d990abc75c2d0c2ebeeef3 (diff)
Add comment explaining why identity check is sufficient.
Diffstat (limited to 'searchcore')
-rw-r--r--searchcore/src/vespa/searchcore/proton/server/feedhandler.cpp5
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);
}