diff options
author | Radoslaw Zarzynski <rzarzyns@redhat.com> | 2019-08-10 10:38:20 +0200 |
---|---|---|
committer | Kefu Chai <kchai@redhat.com> | 2019-08-15 04:47:48 +0200 |
commit | f74318e5c6fd348f3c8938b3f52f40e3475fbbb1 (patch) | |
tree | 4fbc4dfa31183de7db6b9a8e184a5f726d2e4141 /src/cls/cephfs | |
parent | osd/PrimaryLogPG: remove unused "parent" pgls-filter (diff) | |
download | ceph-f74318e5c6fd348f3c8938b3f52f40e3475fbbb1.tar.xz ceph-f74318e5c6fd348f3c8938b3f52f40e3475fbbb1.zip |
osd, osdc: drop the unused outdata feature of PGLSFilter.
Before this commit PGLSFilter interface was offering the outdata
parameter in its filter() method:
filter(..., bufferlist& outdata)
OSD was serializing and appending the bufferlist to response to
CEPH_OSD_OP_PGLS_FILTER and CEPH_OSD_OP_PGNLS_FILTER operations.
At the Objecter's side these extra bits were being parsed and
finally stored in NListContext::extra_info. However, it really
looks this member is not used anywhere.
The commit removes the outdata handling on multiple layers: from
PGLSFilter implementations, through OSD till Objecter.
Signed-off-by: Radoslaw Zarzynski <rzarzyns@redhat.com>
Diffstat (limited to 'src/cls/cephfs')
-rw-r--r-- | src/cls/cephfs/cls_cephfs.cc | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/src/cls/cephfs/cls_cephfs.cc b/src/cls/cephfs/cls_cephfs.cc index 7e3214b9246..3500ae6e497 100644 --- a/src/cls/cephfs/cls_cephfs.cc +++ b/src/cls/cephfs/cls_cephfs.cc @@ -146,12 +146,10 @@ public: ~PGLSCephFSFilter() override {} bool reject_empty_xattr() override { return false; } - bool filter(const hobject_t &obj, bufferlist& xattr_data, - bufferlist& outdata) override; + bool filter(const hobject_t &obj, bufferlist& xattr_data) override; }; -bool PGLSCephFSFilter::filter(const hobject_t &obj, - bufferlist& xattr_data, bufferlist& outdata) +bool PGLSCephFSFilter::filter(const hobject_t &obj, bufferlist& xattr_data) { const std::string need_ending = ".00000000"; const std::string &obj_name = obj.oid.name; |