summaryrefslogtreecommitdiffstats
path: root/searchcore
diff options
context:
space:
mode:
authorHenning Baldersheim <balder@yahoo-inc.com>2019-04-01 14:49:57 +0000
committerHenning Baldersheim <balder@yahoo-inc.com>2019-04-01 14:49:57 +0000
commit992de5d4c9d41813ab05183e7c646be249593340 (patch)
tree33fad6ae484b0930d6397db63807e7bfd858f6f4 /searchcore
parent8fa1b255ad11a3ae6b9fafdb90c1ca3b7144eef5 (diff)
XXHChecksumAggregator -> XXH64ChecksumAggregator and update copyright.
Diffstat (limited to 'searchcore')
-rw-r--r--searchcore/src/tests/proton/documentmetastore/documentmetastore_test.cpp4
-rw-r--r--searchcore/src/vespa/searchcore/proton/bucketdb/checksumaggregator.cpp4
-rw-r--r--searchcore/src/vespa/searchcore/proton/bucketdb/checksumaggregator.h2
-rw-r--r--searchcore/src/vespa/searchcore/proton/bucketdb/checksumaggregators.cpp34
-rw-r--r--searchcore/src/vespa/searchcore/proton/bucketdb/checksumaggregators.h16
5 files changed, 31 insertions, 29 deletions
diff --git a/searchcore/src/tests/proton/documentmetastore/documentmetastore_test.cpp b/searchcore/src/tests/proton/documentmetastore/documentmetastore_test.cpp
index d7d6159af61..7e16094bbf1 100644
--- a/searchcore/src/tests/proton/documentmetastore/documentmetastore_test.cpp
+++ b/searchcore/src/tests/proton/documentmetastore/documentmetastore_test.cpp
@@ -24,7 +24,7 @@ LOG_SETUP("documentmetastore_test");
using namespace document;
using proton::bucketdb::BucketState;
using proton::bucketdb::LegacyChecksumAggregator;
-using proton::bucketdb::XXHChecksumAggregator;
+using proton::bucketdb::XXH64ChecksumAggregator;
using proton::bucketdb::IBucketCreateListener;
using search::AttributeFileSaveTarget;
using search::AttributeGuard;
@@ -845,7 +845,7 @@ TEST("requireThatBasicBucketInfoWorks")
BucketState::setChecksumType(BucketState::ChecksumType::LEGACY);
requireThatBasicBucketInfoWorks<LegacyChecksumAggregator>();
BucketState::setChecksumType(BucketState::ChecksumType::XXHASH64);
- requireThatBasicBucketInfoWorks<XXHChecksumAggregator>();
+ requireThatBasicBucketInfoWorks<XXH64ChecksumAggregator>();
}
TEST("requireThatWeCanRetrieveListOfLidsFromBucketId")
diff --git a/searchcore/src/vespa/searchcore/proton/bucketdb/checksumaggregator.cpp b/searchcore/src/vespa/searchcore/proton/bucketdb/checksumaggregator.cpp
index 738a9a81655..20088fee9c0 100644
--- a/searchcore/src/vespa/searchcore/proton/bucketdb/checksumaggregator.cpp
+++ b/searchcore/src/vespa/searchcore/proton/bucketdb/checksumaggregator.cpp
@@ -1,3 +1,5 @@
+// Copyright 2019 Oath Inc. Licensed under the terms of the Apache 2.0 license. See LICENSE in the project root.
+
#include "checksumaggregators.h"
namespace proton::bucketdb {
@@ -8,7 +10,7 @@ ChecksumAggregator::create(ChecksumType type, BucketChecksum seed) {
case ChecksumType::LEGACY:
return std::make_unique<LegacyChecksumAggregator>(seed);
case ChecksumType::XXHASH64:
- return std::make_unique<XXHChecksumAggregator>(seed);
+ return std::make_unique<XXH64ChecksumAggregator>(seed);
}
return std::unique_ptr<ChecksumAggregator>();
}
diff --git a/searchcore/src/vespa/searchcore/proton/bucketdb/checksumaggregator.h b/searchcore/src/vespa/searchcore/proton/bucketdb/checksumaggregator.h
index fc0b7987756..421cca5c6b5 100644
--- a/searchcore/src/vespa/searchcore/proton/bucketdb/checksumaggregator.h
+++ b/searchcore/src/vespa/searchcore/proton/bucketdb/checksumaggregator.h
@@ -1,4 +1,4 @@
-// Copyright 2017 Yahoo Holdings. Licensed under the terms of the Apache 2.0 license. See LICENSE in the project root.
+// Copyright 2019 Oath Inc. Licensed under the terms of the Apache 2.0 license. See LICENSE in the project root.
#pragma once
diff --git a/searchcore/src/vespa/searchcore/proton/bucketdb/checksumaggregators.cpp b/searchcore/src/vespa/searchcore/proton/bucketdb/checksumaggregators.cpp
index c4b57078fcb..f56d29bb82d 100644
--- a/searchcore/src/vespa/searchcore/proton/bucketdb/checksumaggregators.cpp
+++ b/searchcore/src/vespa/searchcore/proton/bucketdb/checksumaggregators.cpp
@@ -1,4 +1,4 @@
-// Copyright 2017 Yahoo Holdings. Licensed under the terms of the Apache 2.0 license. See LICENSE in the project root.
+// Copyright 2019 Oath Inc. Licensed under the terms of the Apache 2.0 license. See LICENSE in the project root.
#include "checksumaggregators.h"
#include <xxhash.h>
@@ -72,39 +72,39 @@ LegacyChecksumAggregator::empty() const { return _checksum == 0; }
-XXHChecksumAggregator *
-XXHChecksumAggregator::clone() const {
- return new XXHChecksumAggregator(*this);
+XXH64ChecksumAggregator *
+XXH64ChecksumAggregator::clone() const {
+ return new XXH64ChecksumAggregator(*this);
}
-XXHChecksumAggregator &
-XXHChecksumAggregator::addDoc(const GlobalId &gid, const Timestamp &timestamp) {
+XXH64ChecksumAggregator &
+XXH64ChecksumAggregator::addDoc(const GlobalId &gid, const Timestamp &timestamp) {
_checksum ^= compute(gid, timestamp);
return *this;
}
-XXHChecksumAggregator &
-XXHChecksumAggregator::removeDoc(const GlobalId &gid, const Timestamp &timestamp) {
+XXH64ChecksumAggregator &
+XXH64ChecksumAggregator::removeDoc(const GlobalId &gid, const Timestamp &timestamp) {
_checksum ^= compute(gid, timestamp);
return *this;
}
-XXHChecksumAggregator &
-XXHChecksumAggregator::addChecksum(const ChecksumAggregator & rhs) {
- _checksum ^= dynamic_cast<const XXHChecksumAggregator &>(rhs)._checksum;
+XXH64ChecksumAggregator &
+XXH64ChecksumAggregator::addChecksum(const ChecksumAggregator & rhs) {
+ _checksum ^= dynamic_cast<const XXH64ChecksumAggregator &>(rhs)._checksum;
return *this;
}
-XXHChecksumAggregator &
-XXHChecksumAggregator::removeChecksum(const ChecksumAggregator & rhs) {
- _checksum ^= dynamic_cast<const XXHChecksumAggregator &>(rhs)._checksum;
+XXH64ChecksumAggregator &
+XXH64ChecksumAggregator::removeChecksum(const ChecksumAggregator & rhs) {
+ _checksum ^= dynamic_cast<const XXH64ChecksumAggregator &>(rhs)._checksum;
return *this;
}
BucketChecksum
-XXHChecksumAggregator::getChecksum() const {
+XXH64ChecksumAggregator::getChecksum() const {
return BucketChecksum((_checksum >> 32) ^ (_checksum & 0xffffffffL));
}
bool
-XXHChecksumAggregator::empty() const { return _checksum == 0; }
+XXH64ChecksumAggregator::empty() const { return _checksum == 0; }
uint64_t
-XXHChecksumAggregator::compute(const GlobalId &gid, const Timestamp &timestamp) {
+XXH64ChecksumAggregator::compute(const GlobalId &gid, const Timestamp &timestamp) {
char buffer[20];
memcpy(&buffer[0], gid.get(), 12);
reinterpret_cast<uint64_t *>(&buffer[12])[0] = timestamp.getValue();
diff --git a/searchcore/src/vespa/searchcore/proton/bucketdb/checksumaggregators.h b/searchcore/src/vespa/searchcore/proton/bucketdb/checksumaggregators.h
index 19f1de59827..6d6f1b7f842 100644
--- a/searchcore/src/vespa/searchcore/proton/bucketdb/checksumaggregators.h
+++ b/searchcore/src/vespa/searchcore/proton/bucketdb/checksumaggregators.h
@@ -1,4 +1,4 @@
-// Copyright 2017 Yahoo Holdings. Licensed under the terms of the Apache 2.0 license. See LICENSE in the project root.
+// Copyright 2019 Oath Inc. Licensed under the terms of the Apache 2.0 license. See LICENSE in the project root.
#include "checksumaggregator.h"
@@ -24,14 +24,14 @@ private:
/**
* Implementations of the bucket checksums based on XXHASH64.
**/
-class XXHChecksumAggregator : public ChecksumAggregator {
+class XXH64ChecksumAggregator : public ChecksumAggregator {
public:
- XXHChecksumAggregator(BucketChecksum seed) : _checksum(seed) { }
- XXHChecksumAggregator * clone() const override;
- XXHChecksumAggregator & addDoc(const GlobalId &gid, const Timestamp &timestamp) override;
- XXHChecksumAggregator & removeDoc(const GlobalId &gid, const Timestamp &timestamp) override;
- XXHChecksumAggregator & addChecksum(const ChecksumAggregator & rhs) override;
- XXHChecksumAggregator & removeChecksum(const ChecksumAggregator & rhs) override;
+ XXH64ChecksumAggregator(BucketChecksum seed) : _checksum(seed) { }
+ XXH64ChecksumAggregator * clone() const override;
+ XXH64ChecksumAggregator & addDoc(const GlobalId &gid, const Timestamp &timestamp) override;
+ XXH64ChecksumAggregator & removeDoc(const GlobalId &gid, const Timestamp &timestamp) override;
+ XXH64ChecksumAggregator & addChecksum(const ChecksumAggregator & rhs) override;
+ XXH64ChecksumAggregator & removeChecksum(const ChecksumAggregator & rhs) override;
BucketChecksum getChecksum() const override;
bool empty() const override;
private: