From: Jiaying Ren Date: Mon, 19 Jun 2017 15:35:46 +0000 (+0800) Subject: rgw: miis trivial cleanup X-Git-Tag: ses5-milestone9~1^2^2 X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=refs%2Fpull%2F16386%2Fhead;p=ceph.git rgw: miis trivial cleanup + drop outdated & misplaced function description + drop unused func find_placement Signed-off-by: Jiaying Ren --- diff --git a/src/rgw/rgw_rados.cc b/src/rgw/rgw_rados.cc index b116eea2b737..fb25357888f4 100644 --- a/src/rgw/rgw_rados.cc +++ b/src/rgw/rgw_rados.cc @@ -5743,10 +5743,6 @@ void RGWRados::create_bucket_id(string *bucket_id) *bucket_id = buf; } -/** - * create a bucket with name bucket and the given list of attrs - * returns 0 on success, -ERR# otherwise. - */ int RGWRados::create_bucket(RGWUserInfo& owner, rgw_bucket& bucket, const string& zonegroup_id, const string& placement_rule, diff --git a/src/rgw/rgw_rados.h b/src/rgw/rgw_rados.h index 6b996c960bfb..76be577d4040 100644 --- a/src/rgw/rgw_rados.h +++ b/src/rgw/rgw_rados.h @@ -1274,19 +1274,6 @@ struct RGWZoneParams : RGWSystemMetaObj { void decode_json(JSONObj *obj); static void generate_test_instances(list& o); - bool find_placement(const rgw_data_placement_target& placement, string *placement_id) { - for (const auto& pp : placement_pools) { - const RGWZonePlacementInfo& info = pp.second; - if (info.index_pool == placement.index_pool.to_str() && - info.data_pool == placement.data_pool.to_str() && - info.data_extra_pool == placement.data_extra_pool.to_str()) { - *placement_id = pp.first; - return true; - } - } - return false; - } - bool get_placement(const string& placement_id, RGWZonePlacementInfo *placement) const { auto iter = placement_pools.find(placement_id); if (iter == placement_pools.end()) { @@ -2595,10 +2582,6 @@ public: int create_pool(const rgw_pool& pool); - /** - * create a bucket with name bucket and the given list of attrs - * returns 0 on success, -ERR# otherwise. - */ int init_bucket_index(RGWBucketInfo& bucket_info, int num_shards); int select_bucket_placement(RGWUserInfo& user_info, const string& zonegroup_id, const string& rule, string *pselected_rule_name, RGWZonePlacementInfo *rule_info);