diff options
author | Henning Baldersheim <balder@yahoo-inc.com> | 2020-11-25 18:17:27 +0000 |
---|---|---|
committer | Henning Baldersheim <balder@yahoo-inc.com> | 2020-11-26 00:23:06 +0000 |
commit | ee2baa1b001a282bd57318a3f0b8881cdcbc3049 (patch) | |
tree | e1bd88266adb509a9ce4006f7d68cbc59db3c295 /storage/src/tests/storageserver/rpc/caching_rpc_target_resolver_test.cpp | |
parent | e1584673531bc771fa94731da337ce311b4ff7d1 (diff) |
As we have have now removed the expensive Route member we can further compact the message objects.
- Compact StorageMessageAddress to 16 bytes by
- using reference to cluster name.
- Use small enums for protocol and node type.
- Avoid having StorageMessage as separate allocation.
- Avoid default values
Diffstat (limited to 'storage/src/tests/storageserver/rpc/caching_rpc_target_resolver_test.cpp')
-rw-r--r-- | storage/src/tests/storageserver/rpc/caching_rpc_target_resolver_test.cpp | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/storage/src/tests/storageserver/rpc/caching_rpc_target_resolver_test.cpp b/storage/src/tests/storageserver/rpc/caching_rpc_target_resolver_test.cpp index 6291a8ad0dd..8f8eb84b0f2 100644 --- a/storage/src/tests/storageserver/rpc/caching_rpc_target_resolver_test.cpp +++ b/storage/src/tests/storageserver/rpc/caching_rpc_target_resolver_test.cpp @@ -49,6 +49,8 @@ public: } }; +vespalib::string _my_cluster("my_cluster"); + class CachingRpcTargetResolverTest : public ::testing::Test { public: MockMirror mirror; @@ -66,8 +68,8 @@ public: : mirror(), factory(), resolver(mirror, factory, 2), - address_0("my_cluster", NodeType::STORAGE, 5), - address_1("my_cluster", NodeType::DISTRIBUTOR, 7), + address_0(&_my_cluster, NodeType::STORAGE, 5), + address_1(&_my_cluster, NodeType::DISTRIBUTOR, 7), spec_0("tcp/my:41"), spec_1("tcp/my:42"), bucket_id_0(3), |