summaryrefslogtreecommitdiffstats
path: root/searchcore
diff options
context:
space:
mode:
authorHenning Baldersheim <balder@yahoo-inc.com>2020-08-28 13:10:22 +0200
committerGitHub <noreply@github.com>2020-08-28 13:10:22 +0200
commit1bec5243afafe637abafed21600ef11e997cd7b8 (patch)
tree97c4c48b4eb3f5933c8d94a612252c3ad22a8fb8 /searchcore
parente9b01926c98277e489a7ecfb544aaf819b72fe4e (diff)
parent6d26d9a1fb8ccdcc2b930f9ce3561820fb84acae (diff)
Merge pull request #14184 from vespa-engine/balder/style-changes
Style chnages
Diffstat (limited to 'searchcore')
-rw-r--r--searchcore/src/tests/proton/documentdb/feedhandler/feedhandler_test.cpp1
-rw-r--r--searchcore/src/tests/proton/server/feedstates_test.cpp4
-rw-r--r--searchcore/src/vespa/searchcore/proton/server/tlcproxy.cpp3
3 files changed, 2 insertions, 6 deletions
diff --git a/searchcore/src/tests/proton/documentdb/feedhandler/feedhandler_test.cpp b/searchcore/src/tests/proton/documentdb/feedhandler/feedhandler_test.cpp
index 4a01d7ae3e8..18b3a5c5d8e 100644
--- a/searchcore/src/tests/proton/documentdb/feedhandler/feedhandler_test.cpp
+++ b/searchcore/src/tests/proton/documentdb/feedhandler/feedhandler_test.cpp
@@ -23,7 +23,6 @@
#include <vespa/searchcore/proton/server/i_feed_handler_owner.h>
#include <vespa/searchcore/proton/server/ireplayconfig.h>
#include <vespa/searchcore/proton/test/dummy_feed_view.h>
-#include <vespa/searchlib/common/idestructorcallback.h>
#include <vespa/searchlib/index/docbuilder.h>
#include <vespa/searchlib/index/dummyfileheadercontext.h>
#include <vespa/searchlib/transactionlog/translogserver.h>
diff --git a/searchcore/src/tests/proton/server/feedstates_test.cpp b/searchcore/src/tests/proton/server/feedstates_test.cpp
index fd1e24c1f17..15083975824 100644
--- a/searchcore/src/tests/proton/server/feedstates_test.cpp
+++ b/searchcore/src/tests/proton/server/feedstates_test.cpp
@@ -1,7 +1,6 @@
// Copyright 2017 Yahoo Holdings. Licensed under the terms of the Apache 2.0 license. See LICENSE in the project root.
// Unit tests for feedstates.
-
#include <vespa/document/base/documentid.h>
#include <vespa/document/base/testdocrepo.h>
#include <vespa/document/bucket/bucketid.h>
@@ -102,11 +101,10 @@ struct RemoveOperationContext
RemoveOperationContext::RemoveOperationContext(search::SerialNum serial)
: doc_id("id:ns:doctypename::bar"),
op(BucketFactory::getBucketId(doc_id), Timestamp(10), doc_id),
- str(), packet()
+ str(), packet(std::make_unique<Packet>(0xf000))
{
op.serialize(str);
ConstBufferRef buf(str.data(), str.wp());
- packet = std::make_unique<Packet>();
packet->add(Packet::Entry(serial, FeedOperation::REMOVE, buf));
}
RemoveOperationContext::~RemoveOperationContext() = default;
diff --git a/searchcore/src/vespa/searchcore/proton/server/tlcproxy.cpp b/searchcore/src/vespa/searchcore/proton/server/tlcproxy.cpp
index bbd02d7efce..baba74c482c 100644
--- a/searchcore/src/vespa/searchcore/proton/server/tlcproxy.cpp
+++ b/searchcore/src/vespa/searchcore/proton/server/tlcproxy.cpp
@@ -15,9 +15,8 @@ void TlcProxy::commit(search::SerialNum serialNum, search::transactionlog::Type
const vespalib::nbostream &buf, DoneCallback onDone)
{
Packet::Entry entry(serialNum, type, vespalib::ConstBufferRef(buf.data(), buf.size()));
- Packet packet;
+ Packet packet(entry.serializedSize());
packet.add(entry);
- packet.close();
_tlsDirectWriter.commit(_domain, packet, std::move(onDone));
}