bahman/scylla
1
0
Fork 0
Code Issues Pull Requests Actions 2 Packages Projects Releases Wiki Activity

keys: remove with_linearized uses

Browse Source 
There is a variant of `to_hex` that works with `managed_bytes_view`,
no need to linearize.

Tests: unit(dev)

[avi: edit out unneeded std::ref()]

Signed-off-by: Pavel Solodovnikov <pa.solodovnikov@scylladb.com>
Message-Id: <20210828093252.650928-1-pa.solodovnikov@scylladb.com>
This commit is contained in:
Pavel Solodovnikov
2021-08-28 12:32:52 +03:00
committed by Avi Kivity
parent 7f44736939
commit 998dadf479
1 changed files with 2 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
keys.cc
View File

@@ -28,9 +28,7 @@
#include <boost/any.hpp>
std::ostream& operator<<(std::ostream& out, const partition_key& pk) {
return pk.representation().with_linearized([&] (bytes_view v) {
return std::ref(out << "pk{" << to_hex(v) << "}");
});
return out << "pk{" << to_hex(managed_bytes_view(pk.representation())) << "}";
}
template<typename T>
@@ -64,9 +62,7 @@ std::ostream& operator<<(std::ostream& out, const partition_key_view& pk) {
}
std::ostream& operator<<(std::ostream& out, const clustering_key_prefix& ckp) {
return ckp.representation().with_linearized([&] (bytes_view v) {
return std::ref(out << "ckp{" << to_hex(v) << "}");
});
return out << "ckp{" << to_hex(managed_bytes_view(ckp.representation())) << "}";
}
const legacy_compound_view<partition_key_view::c_type>