summaryrefslogtreecommitdiffstats
path: root/src/test/rgw
diff options
context:
space:
mode:
authorCasey Bodley <cbodley@redhat.com>2023-11-22 17:20:26 +0100
committerCasey Bodley <cbodley@redhat.com>2023-11-23 20:47:08 +0100
commit3bf735b4ab78f64e434feec7f11b84a2d9aa1a01 (patch)
tree7fc883b296ace4c31c7da01e4444b901bdef09d8 /src/test/rgw
parentrgw/sal: RGWGetUserStats_CB cleanup (diff)
downloadceph-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.cc2
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;
}