diff options
author | Casey Bodley <cbodley@redhat.com> | 2023-11-22 17:20:26 +0100 |
---|---|---|
committer | Casey Bodley <cbodley@redhat.com> | 2023-11-23 20:47:08 +0100 |
commit | 3bf735b4ab78f64e434feec7f11b84a2d9aa1a01 (patch) | |
tree | 7fc883b296ace4c31c7da01e4444b901bdef09d8 /src/test/rgw | |
parent | rgw/sal: RGWGetUserStats_CB cleanup (diff) | |
download | ceph-3bf735b4ab78f64e434feec7f11b84a2d9aa1a01.tar.xz ceph-3bf735b4ab78f64e434feec7f11b84a2d9aa1a01.zip |
rgw/sal: unify ReadStatsCB for async user/bucket stats
`rgw::sal::Bucket::read_stats_async()` had returned stats as
`map<RGWObjCategory, RGWStorageStats>`, but the only caller in
rgw_quota.cc just summed up the categories into a single instance
of `RGWStorageStats`. moving this summation up into rgw_rados'
`RGWGetBucketStatsContext` allows us to unify these sal callbacks
around `RGWStorageStats` alone
the `RGWGetUserStats_CB`/`RGWGetBucketStats_CB` callbacks had inherited
from `RefCountedObject` and required manual reference counting. switch
to `boost::intrusive_ptr` for scope-based shared ownership
Signed-off-by: Casey Bodley <cbodley@redhat.com>
Diffstat (limited to 'src/test/rgw')
-rw-r--r-- | src/test/rgw/test_rgw_lua.cc | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/test/rgw/test_rgw_lua.cc b/src/test/rgw/test_rgw_lua.cc index be9fa6aed16..dfd2767f550 100644 --- a/src/test/rgw/test_rgw_lua.cc +++ b/src/test/rgw/test_rgw_lua.cc @@ -86,7 +86,7 @@ public: return 0; } - virtual int read_stats_async(const DoutPrefixProvider *dpp, RGWGetUserStats_CB *cb) override { + virtual int read_stats_async(const DoutPrefixProvider *dpp, boost::intrusive_ptr<sal::ReadStatsCB> cb) override { return 0; } |