Merge pull request #4054 from vespa-engine/toregge/remove-dead-code-in-clusterinformation

Remove dead code in distributor cluster information.

2 files changed, 0 insertions, 17 deletions

diff --git a/storage/src/vespa/storage/distributor/clusterinformation.cpp b/storage/src/vespa/storage/distributor/clusterinformation.cpp
index 8e956a1cf61..2e22d7db51d 100644
--- a/storage/src/vespa/storage/distributor/clusterinformation.cpp
+++ b/storage/src/vespa/storage/distributor/clusterinformation.cpp
@@ -7,21 +7,6 @@
 namespace storage::distributor {
 
 bool
-ClusterInformation::ownsBucket(const document::BucketId& bucketId) const
-{
-    try {
-        uint16_t distributor(getDistribution().getIdealDistributorNode(
-                        getClusterState(), bucketId));
-
-        return (getDistributorIndex() == distributor);
-    } catch (lib::TooFewBucketBitsInUseException& e) {
-        return false;
-    } catch (lib::NoDistributorsAvailableException& e) {
-        return false;
-    }
-}
-
-bool
 ClusterInformation::nodeInSameGroupAsSelf(uint16_t otherNode) const
 {
     return (getDistribution().getNodeGraph().getGroupForNode(otherNode)
diff --git a/storage/src/vespa/storage/distributor/clusterinformation.h b/storage/src/vespa/storage/distributor/clusterinformation.h
index 4494b137f89..8b416e26147 100644
--- a/storage/src/vespa/storage/distributor/clusterinformation.h
+++ b/storage/src/vespa/storage/distributor/clusterinformation.h
@@ -32,8 +32,6 @@ public:
 
     virtual const char* getStorageUpStates() const = 0;
 
-    bool ownsBucket(const document::BucketId& bucketId) const;
-
     bool nodeInSameGroupAsSelf(uint16_t otherNode) const;
 
     vespalib::string getDistributionHash() const;