]> git-server-git.apps.pok.os.sepia.ceph.com Git - ceph.git/commitdiff
rgw: Remove straggling `using namespace std` from headers
authorAdam C. Emerson <aemerson@redhat.com>
Sat, 22 Jan 2022 00:54:47 +0000 (19:54 -0500)
committerAdam C. Emerson <aemerson@redhat.com>
Sat, 22 Jan 2022 03:28:18 +0000 (22:28 -0500)
A few crept back in.

Signed-off-by: Adam C. Emerson <aemerson@redhat.com>
12 files changed:
src/rgw/rgw_object_expirer.cc
src/rgw/rgw_object_expirer_core.h
src/rgw/rgw_rados.cc
src/rgw/rgw_sal_dbstore.cc
src/rgw/rgw_sal_dbstore.h
src/rgw/store/dbstore/common/dbstore.h
src/rgw/store/dbstore/common/dbstore_log.h
src/rgw/store/dbstore/dbstore_main.cc
src/rgw/store/dbstore/dbstore_mgr.cc
src/rgw/store/dbstore/dbstore_mgr.h
src/rgw/store/dbstore/sqlite/sqliteDB.cc
src/rgw/store/dbstore/sqlite/sqliteDB.h

index e2e82ad58b12e14d915498185196ef78f0713ce7..1fd940533c510062d89e4144517f500f828012fc 100644 (file)
@@ -55,7 +55,7 @@ int main(const int argc, const char **argv)
 {
   auto args = argv_to_vec(argc, argv);
   if (args.empty()) {
-    cerr << argv[0] << ": -h or --help for usage" << std::endl;
+    std::cerr << argv[0] << ": -h or --help for usage" << std::endl;
     exit(1);
   }
   if (ceph_argparse_need_usage(args)) {
index 68b65653ba47b0dbaa4042f935af4a43732f3e10..c02ca5ea9dffbf11b7dc63f2001dc331acade7ad 100644 (file)
@@ -32,8 +32,6 @@
 #include "rgw_sal.h"
 #include "rgw_sal_rados.h"
 
-using namespace std;
-
 class RGWSI_RADOS;
 class RGWSI_Zone;
 class RGWBucketInfo;
@@ -50,27 +48,27 @@ public:
 
   int objexp_hint_add(const DoutPrefixProvider *dpp, 
                       const ceph::real_time& delete_at,
-                      const string& tenant_name,
-                      const string& bucket_name,
-                      const string& bucket_id,
+                      const std::string& tenant_name,
+                      const std::string& bucket_name,
+                      const std::string& bucket_id,
                       const rgw_obj_index_key& obj_key);
 
   int objexp_hint_list(const DoutPrefixProvider *dpp, 
-                       const string& oid,
+                       const std::string& oid,
                        const ceph::real_time& start_time,
                        const ceph::real_time& end_time,
                        const int max_entries,
-                       const string& marker,
-                       list<cls_timeindex_entry>& entries, /* out */
-                       string *out_marker,                 /* out */
+                       const std::string& marker,
+                       std::list<cls_timeindex_entry>& entries, /* out */
+                       std::string *out_marker,                 /* out */
                        bool *truncated);                   /* out */
 
   int objexp_hint_trim(const DoutPrefixProvider *dpp, 
-                       const string& oid,
+                       const std::string& oid,
                        const ceph::real_time& start_time,
                        const ceph::real_time& end_time,
-                       const string& from_marker,
-                       const string& to_marker);
+                       const std::string& from_marker,
+                       const std::string& to_marker);
 };
 
 class RGWObjectExpirer {
@@ -95,8 +93,8 @@ protected:
     void stop();
 
     CephContext *get_cct() const override;
-    unsigned get_subsys() const;
-    std::ostream& gen_prefix(std::ostream& out) const;
+    unsigned get_subsys() const override;
+    std::ostream& gen_prefix(std::ostream& out) const override;
   };
 
   OEWorker *worker{nullptr};
@@ -114,9 +112,9 @@ public:
 
   int hint_add(const DoutPrefixProvider *dpp, 
                const ceph::real_time& delete_at,
-               const string& tenant_name,
-               const string& bucket_name,
-               const string& bucket_id,
+               const std::string& tenant_name,
+               const std::string& bucket_name,
+               const std::string& bucket_id,
                const rgw_obj_index_key& obj_key) {
     return exp_store.objexp_hint_add(dpp, delete_at, tenant_name, bucket_name,
                                      bucket_id, obj_key);
@@ -132,8 +130,8 @@ public:
                   const std::string& shard,
                   const utime_t& from,
                   const utime_t& to,
-                  const string& from_marker,
-                  const string& to_marker);
+                  const std::string& from_marker,
+                  const std::string& to_marker);
 
   bool process_single_shard(const DoutPrefixProvider *dpp, 
                             const std::string& shard,
index 5fb9e9f3c48b55cae7aca35918aa15f646c953b7..d90b4bcae919de6dd9c67dad944ae7aae0754b66 100644 (file)
@@ -52,8 +52,6 @@
 
 #include "common/Clock.h"
 
-using namespace librados;
-
 #include <string>
 #include <iostream>
 #include <vector>
@@ -101,6 +99,8 @@ using namespace librados;
 #define dout_context g_ceph_context
 #define dout_subsys ceph_subsys_rgw
 
+using namespace std;
+using namespace librados;
 
 static string shadow_ns = "shadow";
 static string default_bucket_index_pool_suffix = "rgw.buckets.index";
index f646d94357dfa1aaa3170be62518108dadc6eb67..947ba8909f9e7edb759ef663ef84287ac91b5c0c 100644 (file)
@@ -28,6 +28,8 @@
 
 #define dout_subsys ceph_subsys_rgw
 
+using namespace std;
+
 namespace rgw::sal {
 
   int DBUser::list_buckets(const DoutPrefixProvider *dpp, const string& marker,
index 49c1bc4340bc24e8bcc5d2bb1507495d7b7a5ae8..c5b85095041c0b1fab96c3e0091fc27024b1b916 100644 (file)
@@ -110,7 +110,7 @@ protected:
       virtual int complete_flush_stats(const DoutPrefixProvider *dpp, optional_yield y) override;
       virtual int read_usage(const DoutPrefixProvider *dpp, uint64_t start_epoch, uint64_t end_epoch, uint32_t max_entries,
           bool* is_truncated, RGWUsageIter& usage_iter,
-          map<rgw_user_bucket, rgw_usage_log_entry>& usage) override;
+          std::map<rgw_user_bucket, rgw_usage_log_entry>& usage) override;
       virtual int trim_usage(const DoutPrefixProvider *dpp, uint64_t start_epoch, uint64_t end_epoch) override;
 
       /* Placeholders */
@@ -205,7 +205,7 @@ protected:
       virtual int try_refresh_info(const DoutPrefixProvider *dpp, ceph::real_time *pmtime) override;
       virtual int read_usage(const DoutPrefixProvider *dpp, uint64_t start_epoch, uint64_t end_epoch, uint32_t max_entries,
           bool *is_truncated, RGWUsageIter& usage_iter,
-          map<rgw_user_bucket, rgw_usage_log_entry>& usage) override;
+          std::map<rgw_user_bucket, rgw_usage_log_entry>& usage) override;
       virtual int trim_usage(const DoutPrefixProvider *dpp, uint64_t start_epoch, uint64_t end_epoch) override;
       virtual int remove_objs_from_index(const DoutPrefixProvider *dpp, std::list<rgw_obj_index_key>& objs_to_unlink) override;
       virtual int check_index(const DoutPrefixProvider *dpp, std::map<RGWObjCategory, RGWStorageStats>& existing_stats, std::map<RGWObjCategory, RGWStorageStats>& calculated_stats) override;
@@ -219,12 +219,12 @@ protected:
                                const std::string& oid, std::optional<std::string> upload_id,
                                ACLOwner owner={}, ceph::real_time mtime=ceph::real_clock::now()) override;
       virtual int list_multiparts(const DoutPrefixProvider *dpp,
-                               const string& prefix,
-                               string& marker,
-                               const string& delim,
+                               const std::string& prefix,
+                               std::string& marker,
+                               const std::string& delim,
                                const int& max_uploads,
-                               vector<std::unique_ptr<MultipartUpload>>& uploads,
-                               map<string, bool> *common_prefixes,
+                               std::vector<std::unique_ptr<MultipartUpload>>& uploads,
+                               std::map<std::string, bool> *common_prefixes,
                                bool *is_truncated) override;
       virtual int abort_multiparts(const DoutPrefixProvider* dpp,
                                   CephContext* cct) override;
@@ -620,13 +620,13 @@ protected:
        const rgw_placement_rule *ptail_placement_rule;
        uint64_t olh_epoch;
     std::unique_ptr<rgw::sal::Object> head_obj;
-    string upload_id;
-    string oid; /* object->name() + "." + "upload_id" + "." + part_num */
+    std::string upload_id;
+    std::string oid; /* object->name() + "." + "upload_id" + "." + part_num */
     std::unique_ptr<rgw::sal::Object> meta_obj;
     DB::Object op_target;
     DB::Object::Write parent_op;
     int part_num;
-    string part_num_str;
+    std::string part_num_str;
     uint64_t total_data_size = 0; /* for total data being uploaded */
     bufferlist head_data;
     bufferlist tail_part_data;
@@ -671,7 +671,7 @@ public:
        * multiple db handles (for eg., one for each tenant),
        * use dbsm->getDB(tenant) */
       DB *db;
-      string luarocks_path;
+      std::string luarocks_path;
       DBZone zone;
       RGWSyncModuleInstanceRef sync_module;
       RGWLC* lc;
@@ -728,13 +728,13 @@ public:
     
       virtual RGWLC* get_rgwlc(void) override;
       virtual RGWCoroutinesManagerRegistry* get_cr_registry() override { return NULL; }
-      virtual int log_usage(const DoutPrefixProvider *dpp, map<rgw_user_bucket, RGWUsageBatch>& usage_info) override;
+      virtual int log_usage(const DoutPrefixProvider *dpp, std::map<rgw_user_bucket, RGWUsageBatch>& usage_info) override;
       virtual int log_op(const DoutPrefixProvider *dpp, std::string& oid, bufferlist& bl) override;
-      virtual int register_to_service_map(const DoutPrefixProvider *dpp, const string& daemon_type,
-          const map<string, string>& meta) override;
+      virtual int register_to_service_map(const DoutPrefixProvider *dpp, const std::string& daemon_type,
+          const std::map<std::string, std::string>& meta) override;
       virtual void get_ratelimit(RGWRateLimitInfo& bucket_ratelimit, RGWRateLimitInfo& user_ratelimit, RGWRateLimitInfo& anon_ratelimit) override;
       virtual void get_quota(RGWQuotaInfo& bucket_quota, RGWQuotaInfo& user_quota) override;
-      virtual int set_buckets_enabled(const DoutPrefixProvider *dpp, vector<rgw_bucket>& buckets, bool enabled) override;
+    virtual int set_buckets_enabled(const DoutPrefixProvider *dpp, std::vector<rgw_bucket>& buckets, bool enabled) override;
       virtual uint64_t get_new_req_id() override { return 0; }
       virtual int get_sync_policy_handler(const DoutPrefixProvider *dpp,
           std::optional<rgw_zone_id> zone,
@@ -742,20 +742,20 @@ public:
           RGWBucketSyncPolicyHandlerRef *phandler,
           optional_yield y) override;
       virtual RGWDataSyncStatusManager* get_data_sync_manager(const rgw_zone_id& source_zone) override;
-      virtual void wakeup_meta_sync_shards(set<int>& shard_ids) override { return; }
-      virtual void wakeup_data_sync_shards(const DoutPrefixProvider *dpp, const rgw_zone_id& source_zone, map<int, set<string> >& shard_ids) override { return; }
+      virtual void wakeup_meta_sync_shards(std::set<int>& shard_ids) override { return; }
+      virtual void wakeup_data_sync_shards(const DoutPrefixProvider *dpp, const rgw_zone_id& source_zone, std::map<int, std::set<std::string>>& shard_ids) override { return; }
       virtual int clear_usage(const DoutPrefixProvider *dpp) override { return 0; }
       virtual int read_all_usage(const DoutPrefixProvider *dpp, uint64_t start_epoch, uint64_t end_epoch,
           uint32_t max_entries, bool *is_truncated,
           RGWUsageIter& usage_iter,
-          map<rgw_user_bucket, rgw_usage_log_entry>& usage) override;
+          std::map<rgw_user_bucket, rgw_usage_log_entry>& usage) override;
       virtual int trim_all_usage(const DoutPrefixProvider *dpp, uint64_t start_epoch, uint64_t end_epoch) override;
       virtual int get_config_key_val(std::string name, bufferlist* bl) override;
       virtual int meta_list_keys_init(const DoutPrefixProvider *dpp, const std::string& section, const std::string& marker, void** phandle) override;
-      virtual int meta_list_keys_next(const DoutPrefixProvider *dpp, void* handle, int max, list<std::string>& keys, bool* truncated) override;
+      virtual int meta_list_keys_next(const DoutPrefixProvider *dpp, void* handle, int max, std::list<std::string>& keys, bool* truncated) override;
       virtual void meta_list_keys_complete(void* handle) override;
       virtual std::string meta_get_marker(void *handle) override;
-      virtual int meta_remove(const DoutPrefixProvider *dpp, string& metadata_key, optional_yield y) override;
+      virtual int meta_remove(const DoutPrefixProvider *dpp, std::string& metadata_key, optional_yield y) override;
 
       virtual const RGWSyncModuleInstanceRef& get_sync_module() { return sync_module; }
       virtual std::string get_host_id() { return ""; }
@@ -772,11 +772,11 @@ public:
           optional_yield y,
           const std::string& path_prefix,
           const std::string& tenant,
-          vector<std::unique_ptr<RGWRole>>& roles) override;
+          std::vector<std::unique_ptr<RGWRole>>& roles) override;
       virtual std::unique_ptr<RGWOIDCProvider> get_oidc_provider() override;
       virtual int get_oidc_providers(const DoutPrefixProvider *dpp,
           const std::string& tenant,
-          vector<std::unique_ptr<RGWOIDCProvider>>& providers) override;
+          std::vector<std::unique_ptr<RGWOIDCProvider>>& providers) override;
       virtual std::unique_ptr<Writer> get_append_writer(const DoutPrefixProvider *dpp,
                                  optional_yield y,
                                  std::unique_ptr<rgw::sal::Object> _head_obj,
@@ -814,7 +814,7 @@ public:
       void setDB(DB * st) { db = st; }
       DB *getDB(void) { return db; }
 
-      DB *getDB(string tenant) { return dbsm->getDB(tenant, false); }
+      DB *getDB(std::string tenant) { return dbsm->getDB(tenant, false); }
   };
 
 } } // namespace rgw::sal
index 9bb61dca7a972068ace2ad62aa1da0b53d5412e1..2fe323d75ffad288cbea1d96a213c66b5598c657 100644 (file)
@@ -24,8 +24,6 @@
 #include "rgw/rgw_obj_manifest.h"
 #include "rgw/rgw_multi.h"
 
-using namespace std;
-
 namespace rgw { namespace store {
 
 class DB;
@@ -44,9 +42,9 @@ struct DBOpBucketInfo {
   obj_version bucket_version;
   ceph::real_time mtime;
   // used for list query
-  string min_marker;
-  string max_marker;
-  list<RGWBucketEnt> list_entries;
+  std::string min_marker;
+  std::string max_marker;
+  std::list<RGWBucketEnt> list_entries;
 };
 
 struct DBOpObjectInfo {
@@ -67,15 +65,15 @@ struct DBOpObjectInfo {
   uint64_t versioned_epoch;
 
   /* from state.manifest (RGWObjManifest) */
-  map<uint64_t, RGWObjManifestPart> objs;
+  std::map<uint64_t, RGWObjManifestPart> objs;
   uint64_t head_size{0};
   rgw_placement_rule head_placement_rule;
   uint64_t max_head_size{0};
-  string prefix;
+  std::string prefix;
   rgw_bucket_placement tail_placement; /* might be different than the original bucket,
                                           as object might have been copied across pools */
-  map<uint64_t, RGWObjManifestRule> rules;
-  string tail_instance; /* tail object's instance */
+  std::map<uint64_t, RGWObjManifestRule> rules;
+  std::string tail_instance; /* tail object's instance */
 
 
   /* Obj's omap <key,value> store */
@@ -86,9 +84,9 @@ struct DBOpObjectInfo {
   std::list<RGWUploadPartInfo> mp_parts;
 
   bufferlist head_data;
-  string min_marker;
-  string max_marker;
-  list<rgw_bucket_dir_entry> list_entries;
+  std::string min_marker;
+  std::string max_marker;
+  std::list<rgw_bucket_dir_entry> list_entries;
   /* Below used to update mp_parts obj name
    * from meta object to src object on completion */
   rgw_obj_key new_obj_key;
@@ -97,34 +95,34 @@ struct DBOpObjectInfo {
 struct DBOpObjectDataInfo {
   RGWObjState state;
   uint64_t part_num;
-  string multipart_part_str;
+  std::string multipart_part_str;
   uint64_t offset;
   uint64_t size;
   bufferlist data{};
 };
 
 struct DBOpLCHeadInfo {
-  string index;
+  std::string index;
   rgw::sal::Lifecycle::LCHead head;
 };
 
 struct DBOpLCEntryInfo {
-  string index;
+  std::string index;
   rgw::sal::Lifecycle::LCEntry entry;
   // used for list query
-  string min_marker;
-  list<rgw::sal::Lifecycle::LCEntry> list_entries;
+  std::string min_marker;
+  std::list<rgw::sal::Lifecycle::LCEntry> list_entries;
 };
 
 struct DBOpInfo {
-  string name; // Op name
+  std::string name; // Op name
   /* Support only single access_key for now. So store
    * it separately as primary access_key_id & secret to
    * be able to query easily.
    *
    * XXX: Swift keys and subuser not supported for now */
   DBOpUserInfo user;
-  string query_str;
+  std::string query_str;
   DBOpBucketInfo bucket;
   DBOpObjectInfo obj;
   DBOpObjectDataInfo obj_data;
@@ -137,19 +135,19 @@ struct DBOpParams {
   CephContext *cct;
 
   /* Tables */
-  string user_table;
-  string bucket_table;
-  string object_table;
+  std::string user_table;
+  std::string bucket_table;
+  std::string object_table;
 
   /* Ops*/
   DBOpInfo op;
 
   /* Below are subject to change */
-  string objectdata_table;
-  string quota_table;
-  string lc_head_table;
-  string lc_entry_table;
-  string obj;
+  std::string objectdata_table;
+  std::string quota_table;
+  std::string lc_head_table;
+  std::string lc_entry_table;
+  std::string obj;
 };
 
 /* Used for prepared schemas.
@@ -163,11 +161,11 @@ struct DBOpParams {
  * to get the right index of each param.
  */
 struct DBOpUserPrepareInfo {
-  string user_id = ":user_id";
-  string tenant = ":tenant";
-  string ns = ":ns";
-  string display_name = ":display_name";
-  string user_email = ":user_email";
+  std::string user_id = ":user_id";
+  std::string tenant = ":tenant";
+  std::string ns = ":ns";
+  std::string display_name = ":display_name";
+  std::string user_email = ":user_email";
   /* Support only single access_key for now. So store
    * it separately as primary access_key_id & secret to
    * be able to query easily.
@@ -175,169 +173,169 @@ struct DBOpUserPrepareInfo {
    * In future, when need to support & query from multiple
    * access keys, better to maintain them in a separate table.
    */
-  string access_keys_id = ":access_keys_id";
-  string access_keys_secret = ":access_keys_secret";
-  string access_keys = ":access_keys";
-  string swift_keys = ":swift_keys";
-  string subusers = ":subusers";
-  string suspended = ":suspended";
-  string max_buckets = ":max_buckets";
-  string op_mask = ":op_mask";
-  string user_caps = ":user_caps";
-  string admin = ":admin";
-  string system = ":system";
-  string placement_name = ":placement_name";
-  string placement_storage_class = ":placement_storage_class";
-  string placement_tags = ":placement_tags";
-  string bucket_quota = ":bucket_quota";
-  string temp_url_keys = ":temp_url_keys";
-  string user_quota = ":user_quota";
-  string type = ":type";
-  string mfa_ids = ":mfa_ids";
-  string assumed_role_arn = ":assumed_role_arn";
-  string user_attrs = ":user_attrs";
-  string user_ver = ":user_vers";
-  string user_ver_tag = ":user_ver_tag";
+  std::string access_keys_id = ":access_keys_id";
+  std::string access_keys_secret = ":access_keys_secret";
+  std::string access_keys = ":access_keys";
+  std::string swift_keys = ":swift_keys";
+  std::string subusers = ":subusers";
+  std::string suspended = ":suspended";
+  std::string max_buckets = ":max_buckets";
+  std::string op_mask = ":op_mask";
+  std::string user_caps = ":user_caps";
+  std::string admin = ":admin";
+  std::string system = ":system";
+  std::string placement_name = ":placement_name";
+  std::string placement_storage_class = ":placement_storage_class";
+  std::string placement_tags = ":placement_tags";
+  std::string bucket_quota = ":bucket_quota";
+  std::string temp_url_keys = ":temp_url_keys";
+  std::string user_quota = ":user_quota";
+  std::string type = ":type";
+  std::string mfa_ids = ":mfa_ids";
+  std::string assumed_role_arn = ":assumed_role_arn";
+  std::string user_attrs = ":user_attrs";
+  std::string user_ver = ":user_vers";
+  std::string user_ver_tag = ":user_ver_tag";
 };
 
 struct DBOpBucketPrepareInfo {
-  string bucket_name = ":bucket_name";
-  string tenant = ":tenant";
-  string marker = ":marker";
-  string bucket_id = ":bucket_id";
-  string size = ":size";
-  string size_rounded = ":size_rounded";
-  string creation_time = ":creation_time";
-  string count = ":count";
-  string placement_name = ":placement_name";
-  string placement_storage_class = ":placement_storage_class";
+  std::string bucket_name = ":bucket_name";
+  std::string tenant = ":tenant";
+  std::string marker = ":marker";
+  std::string bucket_id = ":bucket_id";
+  std::string size = ":size";
+  std::string size_rounded = ":size_rounded";
+  std::string creation_time = ":creation_time";
+  std::string count = ":count";
+  std::string placement_name = ":placement_name";
+  std::string placement_storage_class = ":placement_storage_class";
   /* ownerid - maps to DBOpUserPrepareInfo */
-  string flags = ":flags";
-  string zonegroup = ":zonegroup";
-  string has_instance_obj = ":has_instance_obj";
-  string quota = ":quota";
-  string requester_pays = ":requester_pays";
-  string has_website = ":has_website";
-  string website_conf = ":website_conf";
-  string swift_versioning = ":swift_versioning";
-  string swift_ver_location = ":swift_ver_location";
-  string mdsearch_config = ":mdsearch_config";
-  string new_bucket_instance_id = ":new_bucket_instance_id";
-  string obj_lock = ":obj_lock";
-  string sync_policy_info_groups = ":sync_policy_info_groups";
-  string bucket_attrs = ":bucket_attrs";
-  string bucket_ver = ":bucket_vers";
-  string bucket_ver_tag = ":bucket_ver_tag";
-  string mtime = ":mtime";
-  string min_marker = ":min_marker";
-  string max_marker = ":max_marker";
+  std::string flags = ":flags";
+  std::string zonegroup = ":zonegroup";
+  std::string has_instance_obj = ":has_instance_obj";
+  std::string quota = ":quota";
+  std::string requester_pays = ":requester_pays";
+  std::string has_website = ":has_website";
+  std::string website_conf = ":website_conf";
+  std::string swift_versioning = ":swift_versioning";
+  std::string swift_ver_location = ":swift_ver_location";
+  std::string mdsearch_config = ":mdsearch_config";
+  std::string new_bucket_instance_id = ":new_bucket_instance_id";
+  std::string obj_lock = ":obj_lock";
+  std::string sync_policy_info_groups = ":sync_policy_info_groups";
+  std::string bucket_attrs = ":bucket_attrs";
+  std::string bucket_ver = ":bucket_vers";
+  std::string bucket_ver_tag = ":bucket_ver_tag";
+  std::string mtime = ":mtime";
+  std::string min_marker = ":min_marker";
+  std::string max_marker = ":max_marker";
 };
 
 struct DBOpObjectPrepareInfo {
-  string obj_name = ":obj_name";
-  string obj_instance = ":obj_instance";
-  string obj_ns  = ":obj_ns";
-  string acls = ":acls";
-  string index_ver = ":index_ver";
-  string tag = ":tag";
-  string flags = ":flags";
-  string versioned_epoch = ":versioned_epoch";
-  string obj_category = ":obj_category";
-  string etag = ":etag";
-  string owner = ":owner";
-  string owner_display_name = ":owner_display_name";
-  string storage_class = ":storage_class";
-  string appendable = ":appendable";
-  string content_type = ":content_type";
-  string index_hash_source = ":index_hash_source";
-  string obj_size = ":obj_size";
-  string accounted_size = ":accounted_size";
-  string mtime = ":mtime";
-  string epoch = ":epoch";
-  string obj_tag = ":obj_tag";
-  string tail_tag = ":tail_tag";
-  string write_tag = ":write_tag";
-  string fake_tag = ":fake_tag";
-  string shadow_obj = ":shadow_obj";
-  string has_data = ":has_data";
-  string is_olh = ":is_ols";
-  string olh_tag = ":olh_tag";
-  string pg_ver = ":pg_ver";
-  string zone_short_id = ":zone_short_id";
-  string obj_version = ":obj_version";
-  string obj_version_tag = ":obj_version_tag";
-  string obj_attrs = ":obj_attrs";
-  string head_size = ":head_size";
-  string max_head_size = ":max_head_size";
-  string prefix = ":prefix";
-  string tail_instance = ":tail_instance";
-  string head_placement_rule_name = ":head_placement_rule_name";
-  string head_placement_storage_class  = ":head_placement_storage_class";
-  string tail_placement_rule_name = ":tail_placement_rule_name";
-  string tail_placement_storage_class  = ":tail_placement_storage_class";
-  string manifest_part_objs = ":manifest_part_objs";
-  string manifest_part_rules = ":manifest_part_rules";
-  string omap = ":omap";
-  string is_multipart = ":is_multipart";
-  string mp_parts = ":mp_parts";
-  string head_data = ":head_data";
-  string min_marker = ":min_marker";
-  string max_marker = ":max_marker";
+  std::string obj_name = ":obj_name";
+  std::string obj_instance = ":obj_instance";
+  std::string obj_ns  = ":obj_ns";
+  std::string acls = ":acls";
+  std::string index_ver = ":index_ver";
+  std::string tag = ":tag";
+  std::string flags = ":flags";
+  std::string versioned_epoch = ":versioned_epoch";
+  std::string obj_category = ":obj_category";
+  std::string etag = ":etag";
+  std::string owner = ":owner";
+  std::string owner_display_name = ":owner_display_name";
+  std::string storage_class = ":storage_class";
+  std::string appendable = ":appendable";
+  std::string content_type = ":content_type";
+  std::string index_hash_source = ":index_hash_source";
+  std::string obj_size = ":obj_size";
+  std::string accounted_size = ":accounted_size";
+  std::string mtime = ":mtime";
+  std::string epoch = ":epoch";
+  std::string obj_tag = ":obj_tag";
+  std::string tail_tag = ":tail_tag";
+  std::string write_tag = ":write_tag";
+  std::string fake_tag = ":fake_tag";
+  std::string shadow_obj = ":shadow_obj";
+  std::string has_data = ":has_data";
+  std::string is_olh = ":is_ols";
+  std::string olh_tag = ":olh_tag";
+  std::string pg_ver = ":pg_ver";
+  std::string zone_short_id = ":zone_short_id";
+  std::string obj_version = ":obj_version";
+  std::string obj_version_tag = ":obj_version_tag";
+  std::string obj_attrs = ":obj_attrs";
+  std::string head_size = ":head_size";
+  std::string max_head_size = ":max_head_size";
+  std::string prefix = ":prefix";
+  std::string tail_instance = ":tail_instance";
+  std::string head_placement_rule_name = ":head_placement_rule_name";
+  std::string head_placement_storage_class  = ":head_placement_storage_class";
+  std::string tail_placement_rule_name = ":tail_placement_rule_name";
+  std::string tail_placement_storage_class  = ":tail_placement_storage_class";
+  std::string manifest_part_objs = ":manifest_part_objs";
+  std::string manifest_part_rules = ":manifest_part_rules";
+  std::string omap = ":omap";
+  std::string is_multipart = ":is_multipart";
+  std::string mp_parts = ":mp_parts";
+  std::string head_data = ":head_data";
+  std::string min_marker = ":min_marker";
+  std::string max_marker = ":max_marker";
   /* Below used to update mp_parts obj name
    * from meta object to src object on completion */
-  string new_obj_name = ":new_obj_name";
-  string new_obj_instance = ":new_obj_instance";
-  string new_obj_ns  = ":new_obj_ns";
+  std::string new_obj_name = ":new_obj_name";
+  std::string new_obj_instance = ":new_obj_instance";
+  std::string new_obj_ns  = ":new_obj_ns";
 };
 
 struct DBOpObjectDataPrepareInfo {
-  string part_num = ":part_num";
-  string offset = ":offset";
-  string data = ":data";
-  string size = ":size";
-  string multipart_part_str = ":multipart_part_str";
+  std::string part_num = ":part_num";
+  std::string offset = ":offset";
+  std::string data = ":data";
+  std::string size = ":size";
+  std::string multipart_part_str = ":multipart_part_str";
 };
 
 struct DBOpLCEntryPrepareInfo {
-  string index = ":index";
-  string bucket_name = ":bucket_name";
-  string start_time = ":start_time";
-  string status = ":status";
-  string min_marker = ":min_marker";
+  std::string index = ":index";
+  std::string bucket_name = ":bucket_name";
+  std::string start_time = ":start_time";
+  std::string status = ":status";
+  std::string min_marker = ":min_marker";
 };
 
 struct DBOpLCHeadPrepareInfo {
-  string index = ":index";
-  string start_date = ":start_date";
-  string marker = ":marker";
+  std::string index = ":index";
+  std::string start_date = ":start_date";
+  std::string marker = ":marker";
 };
 
 struct DBOpPrepareInfo {
   DBOpUserPrepareInfo user;
-  string query_str = ":query_str";
+  std::string query_str = ":query_str";
   DBOpBucketPrepareInfo bucket;
   DBOpObjectPrepareInfo obj;
   DBOpObjectDataPrepareInfo obj_data;
   DBOpLCHeadPrepareInfo lc_head;
   DBOpLCEntryPrepareInfo lc_entry;
-  string list_max_count = ":list_max_count";
+  std::string list_max_count = ":list_max_count";
 };
 
 struct DBOpPrepareParams {
   /* Tables */
-  string user_table;
-  string bucket_table;
-  string object_table;
+  std::string user_table;
+  std::string bucket_table;
+  std::string object_table;
 
   /* Ops */
   DBOpPrepareInfo op;
 
 
   /* below subject to change */
-  string objectdata_table;
-  string quota_table;
-  string lc_head_table;
-  string lc_entry_table;
+  std::string objectdata_table;
+  std::string quota_table;
+  std::string lc_head_table;
+  std::string lc_entry_table;
 };
 
 struct DBOps {
@@ -374,13 +372,13 @@ class ObjectOp {
     class GetObjectDataOp *GetObjectData;
     class DeleteObjectDataOp *DeleteObjectData;
 
-    virtual int InitializeObjectOps(string db_name, const DoutPrefixProvider *dpp) { return 0; }
+    virtual int InitializeObjectOps(std::string db_name, const DoutPrefixProvider *dpp) { return 0; }
     virtual int FreeObjectOps(const DoutPrefixProvider *dpp) { return 0; }
 };
 
 class DBOp {
   private:
-    const string CreateUserTableQ =
+    const std::string CreateUserTableQ =
       /* Corresponds to rgw::sal::User
        *
        * For now only UserID is made Primary key.
@@ -426,7 +424,7 @@ class DBOp {
       UserVersionTag TEXT,      \
       PRIMARY KEY (UserID) \n);";
 
-    const string CreateBucketTableQ =
+    const std::string CreateBucketTableQ =
       /* Corresponds to rgw::sal::Bucket
        *
        *  For now only BucketName is made Primary key.
@@ -481,7 +479,7 @@ class DBOp {
       FOREIGN KEY (OwnerID) \
       REFERENCES '{}' (UserID) ON DELETE CASCADE ON UPDATE CASCADE \n);";
 
-    const string CreateObjectTableQ =
+    const std::string CreateObjectTableQ =
       /* Corresponds to rgw::sal::Object
        *
        *  For now only BucketName, ObjName is made Primary key.
@@ -548,11 +546,11 @@ class DBOp {
       ObjAttrs    BLOB,   \
       HeadSize    INTEGER,    \
       MaxHeadSize    INTEGER,    \
-      Prefix      String, \
-      TailInstance    String, \
-      HeadPlacementRuleName   String, \
+      Prefix      std::string, \
+      TailInstance    std::string, \
+      HeadPlacementRuleName   std::string, \
       HeadPlacementRuleStorageClass String, \
-      TailPlacementRuleName   String, \
+      TailPlacementRuleName   std::string, \
       TailPlacementStorageClass String, \
       ManifestPartObjs    BLOB,   \
       ManifestPartRules   BLOB,   \
@@ -564,7 +562,7 @@ class DBOp {
       FOREIGN KEY (BucketName) \
       REFERENCES '{}' (BucketName) ON DELETE CASCADE ON UPDATE CASCADE \n);";
 
-    const string CreateObjectDataTableQ =
+    const std::string CreateObjectDataTableQ =
       /* Extra field 'MultipartPartStr' added which signifies multipart
        * <uploadid + partnum>. For regular object, it is '0.0'
        *
@@ -588,7 +586,7 @@ class DBOp {
       FOREIGN KEY (BucketName, ObjName, ObjInstance) \
       REFERENCES '{}' (BucketName, ObjName, ObjInstance) ON DELETE CASCADE ON UPDATE CASCADE \n);";
 
-    const string CreateQuotaTableQ =
+    const std::string CreateQuotaTableQ =
       "CREATE TABLE IF NOT EXISTS '{}' ( \
       QuotaID INTEGER PRIMARY KEY AUTOINCREMENT UNIQUE , \
       MaxSizeSoftThreshold INTEGER ,   \
@@ -598,7 +596,7 @@ class DBOp {
       Enabled Boolean ,                \
       CheckOnRaw Boolean \n);";
 
-    const string CreateLCEntryTableQ =
+    const std::string CreateLCEntryTableQ =
       "CREATE TABLE IF NOT EXISTS '{}' ( \
       LCIndex  TEXT NOT NULL , \
       BucketName TEXT NOT NULL , \
@@ -608,22 +606,22 @@ class DBOp {
       FOREIGN KEY (BucketName) \
       REFERENCES '{}' (BucketName) ON DELETE CASCADE ON UPDATE CASCADE \n);";
 
-    const string CreateLCHeadTableQ =
+    const std::string CreateLCHeadTableQ =
       "CREATE TABLE IF NOT EXISTS '{}' ( \
       LCIndex  TEXT NOT NULL , \
       Marker TEXT , \
       StartDate  INTEGER , \
       PRIMARY KEY (LCIndex) \n);";
 
-    const string DropQ = "DROP TABLE IF EXISTS '{}'";
-    const string ListAllQ = "SELECT  * from '{}'";
+    const std::string DropQ = "DROP TABLE IF EXISTS '{}'";
+    const std::string ListAllQ = "SELECT  * from '{}'";
 
   public:
     DBOp() {}
     virtual ~DBOp() {}
     std::mutex mtx; // to protect prepared stmt
 
-    string CreateTableSchema(string type, DBOpParams *params) {
+    std::string CreateTableSchema(std::string type, DBOpParams *params) {
       if (!type.compare("User"))
         return fmt::format(CreateUserTableQ.c_str(),
             params->user_table.c_str());
@@ -655,10 +653,10 @@ class DBOp {
       return NULL;
     }
 
-    string DeleteTableSchema(string table) {
+    std::string DeleteTableSchema(std::string table) {
       return fmt::format(DropQ.c_str(), table.c_str());
     }
-    string ListTableSchema(string table) {
+    std::string ListTableSchema(std::string table) {
       return fmt::format(ListAllQ.c_str(), table.c_str());
     }
 
@@ -680,7 +678,7 @@ class InsertUserOp : virtual public DBOp {
      * For now using INSERT or REPLACE. If required of updating existing
      * record, will use another query.
      */
-    const string Query = "INSERT OR REPLACE INTO '{}'  \
+    const std::string Query = "INSERT OR REPLACE INTO '{}'     \
                           (UserID, Tenant, NS, DisplayName, UserEmail, \
                            AccessKeysID, AccessKeysSecret, AccessKeys, SwiftKeys,\
                            SubUsers, Suspended, MaxBuckets, OpMask, UserCaps, Admin, \
@@ -693,7 +691,7 @@ class InsertUserOp : virtual public DBOp {
   public:
     virtual ~InsertUserOp() {}
 
-    string Schema(DBOpPrepareParams &params) {
+    std::string Schema(DBOpPrepareParams &params) {
       return fmt::format(Query.c_str(), params.user_table.c_str(),
           params.op.user.user_id.c_str(), params.op.user.tenant, params.op.user.ns,
           params.op.user.display_name, params.op.user.user_email,
@@ -714,13 +712,13 @@ class InsertUserOp : virtual public DBOp {
 
 class RemoveUserOp: virtual public DBOp {
   private:
-    const string Query =
+    const std::string Query =
       "DELETE from '{}' where UserID = {}";
 
   public:
     virtual ~RemoveUserOp() {}
 
-    string Schema(DBOpPrepareParams &params) {
+    std::string Schema(DBOpPrepareParams &params) {
       return fmt::format(Query.c_str(), params.user_table.c_str(),
           params.op.user.user_id.c_str());
     }
@@ -730,7 +728,7 @@ class GetUserOp: virtual public DBOp {
   private:
     /* If below query columns are updated, make sure to update the indexes
      * in list_user() cbk in sqliteDB.cc */
-    const string Query = "SELECT \
+    const std::string Query = "SELECT \
                           UserID, Tenant, NS, DisplayName, UserEmail, \
                           AccessKeysID, AccessKeysSecret, AccessKeys, SwiftKeys,\
                           SubUsers, Suspended, MaxBuckets, OpMask, UserCaps, Admin, \
@@ -738,7 +736,7 @@ class GetUserOp: virtual public DBOp {
                           BucketQuota, TempURLKeys, UserQuota, Type, MfaIDs, AssumedRoleARN, \
                           UserAttrs, UserVersion, UserVersionTag from '{}' where UserID = {}";
 
-    const string QueryByEmail = "SELECT \
+    const std::string QueryByEmail = "SELECT \
                                  UserID, Tenant, NS, DisplayName, UserEmail, \
                                  AccessKeysID, AccessKeysSecret, AccessKeys, SwiftKeys,\
                                  SubUsers, Suspended, MaxBuckets, OpMask, UserCaps, Admin, \
@@ -746,7 +744,7 @@ class GetUserOp: virtual public DBOp {
                                  BucketQuota, TempURLKeys, UserQuota, Type, MfaIDs, AssumedRoleARN, \
                                  UserAttrs, UserVersion, UserVersionTag from '{}' where UserEmail = {}";
 
-    const string QueryByAccessKeys = "SELECT \
+    const std::string QueryByAccessKeys = "SELECT \
                                       UserID, Tenant, NS, DisplayName, UserEmail, \
                                       AccessKeysID, AccessKeysSecret, AccessKeys, SwiftKeys,\
                                       SubUsers, Suspended, MaxBuckets, OpMask, UserCaps, Admin, \
@@ -754,7 +752,7 @@ class GetUserOp: virtual public DBOp {
                                       BucketQuota, TempURLKeys, UserQuota, Type, MfaIDs, AssumedRoleARN, \
                                       UserAttrs, UserVersion, UserVersionTag from '{}' where AccessKeysID = {}";
 
-    const string QueryByUserID = "SELECT \
+    const std::string QueryByUserID = "SELECT \
                                   UserID, Tenant, NS, DisplayName, UserEmail, \
                                   AccessKeysID, AccessKeysSecret, AccessKeys, SwiftKeys,\
                                   SubUsers, Suspended, MaxBuckets, OpMask, UserCaps, Admin, \
@@ -766,7 +764,7 @@ class GetUserOp: virtual public DBOp {
   public:
     virtual ~GetUserOp() {}
 
-    string Schema(DBOpPrepareParams &params) {
+    std::string Schema(DBOpPrepareParams &params) {
       if (params.op.query_str == "email") {
         return fmt::format(QueryByEmail.c_str(), params.user_table.c_str(),
             params.op.user.user_email.c_str());
@@ -789,7 +787,7 @@ class GetUserOp: virtual public DBOp {
 
 class InsertBucketOp: virtual public DBOp {
   private:
-    const string Query =
+    const std::string Query =
       "INSERT OR REPLACE INTO '{}' \
       (BucketName, Tenant, Marker, BucketID, Size, SizeRounded, CreationTime, \
        Count, PlacementName, PlacementStorageClass, OwnerID, Flags, Zonegroup, \
@@ -804,7 +802,7 @@ class InsertBucketOp: virtual public DBOp {
   public:
     virtual ~InsertBucketOp() {}
 
-    string Schema(DBOpPrepareParams &params) {
+    std::string Schema(DBOpPrepareParams &params) {
       return fmt::format(Query.c_str(), params.bucket_table.c_str(),
           params.op.bucket.bucket_name, params.op.bucket.tenant,
           params.op.bucket.marker, params.op.bucket.bucket_id,
@@ -826,7 +824,7 @@ class InsertBucketOp: virtual public DBOp {
 class UpdateBucketOp: virtual public DBOp {
   private:
     // Updates Info, Mtime, Version
-    const string InfoQuery =
+    const std::string InfoQuery =
       "UPDATE '{}' SET Tenant = {}, Marker = {}, BucketID = {}, CreationTime = {}, \
       Count = {}, PlacementName = {}, PlacementStorageClass = {}, OwnerID = {}, Flags = {}, \
       Zonegroup = {}, HasInstanceObj = {}, Quota = {}, RequesterPays = {}, HasWebsite = {}, \
@@ -834,17 +832,17 @@ class UpdateBucketOp: virtual public DBOp {
       NewBucketInstanceID = {}, ObjectLock = {}, SyncPolicyInfoGroups = {}, \
       BucketVersion = {}, Mtime = {} WHERE BucketName = {}";
     // Updates Attrs, OwnerID, Mtime, Version
-    const string AttrsQuery =
+    const std::string AttrsQuery =
       "UPDATE '{}' SET OwnerID = {}, BucketAttrs = {}, Mtime = {}, BucketVersion = {} \
       WHERE BucketName = {}";
     // Updates OwnerID, CreationTime, Mtime, Version
-    const string OwnerQuery =
+    const std::string OwnerQuery =
       "UPDATE '{}' SET OwnerID = {}, CreationTime = {}, Mtime = {}, BucketVersion = {} WHERE BucketName = {}";
 
   public:
     virtual ~UpdateBucketOp() {}
 
-    string Schema(DBOpPrepareParams &params) {
+    std::string Schema(DBOpPrepareParams &params) {
       if (params.op.query_str == "info") {
         return fmt::format(InfoQuery.c_str(), params.bucket_table.c_str(),
             params.op.bucket.tenant, params.op.bucket.marker, params.op.bucket.bucket_id,
@@ -878,13 +876,13 @@ class UpdateBucketOp: virtual public DBOp {
 
 class RemoveBucketOp: virtual public DBOp {
   private:
-    const string Query =
+    const std::string Query =
       "DELETE from '{}' where BucketName = {}";
 
   public:
     virtual ~RemoveBucketOp() {}
 
-    string Schema(DBOpPrepareParams &params) {
+    std::string Schema(DBOpPrepareParams &params) {
       return fmt::format(Query.c_str(), params.bucket_table.c_str(),
           params.op.bucket.bucket_name.c_str());
     }
@@ -892,7 +890,7 @@ class RemoveBucketOp: virtual public DBOp {
 
 class GetBucketOp: virtual public DBOp {
   private:
-    const string Query = "SELECT  \
+    const std::string Query = "SELECT  \
                           BucketName, BucketTable.Tenant, Marker, BucketID, Size, SizeRounded, CreationTime, \
                           Count, BucketTable.PlacementName, BucketTable.PlacementStorageClass, OwnerID, Flags, Zonegroup, \
                           HasInstanceObj, Quota, RequesterPays, HasWebsite, WebsiteConf, \
@@ -904,7 +902,7 @@ class GetBucketOp: virtual public DBOp {
   public:
     virtual ~GetBucketOp() {}
 
-    string Schema(DBOpPrepareParams &params) {
+    std::string Schema(DBOpPrepareParams &params) {
       //return fmt::format(Query.c_str(), params.op.bucket.bucket_name.c_str(),
       //          params.bucket_table.c_str(), params.user_table.c_str());
       return fmt::format(Query.c_str(),
@@ -917,7 +915,7 @@ class ListUserBucketsOp: virtual public DBOp {
   private:
     // once we have stats also stored, may have to update this query to join
     // these two tables.
-    const string Query = "SELECT  \
+    const std::string Query = "SELECT  \
                           BucketName, Tenant, Marker, BucketID, Size, SizeRounded, CreationTime, \
                           Count, PlacementName, PlacementStorageClass, OwnerID, Flags, Zonegroup, \
                           HasInstanceObj, Quota, RequesterPays, HasWebsite, WebsiteConf, \
@@ -929,7 +927,7 @@ class ListUserBucketsOp: virtual public DBOp {
   public:
     virtual ~ListUserBucketsOp() {}
 
-    string Schema(DBOpPrepareParams &params) {
+    std::string Schema(DBOpPrepareParams &params) {
       return fmt::format(Query.c_str(), params.bucket_table.c_str(),
           params.op.user.user_id.c_str(), params.op.bucket.min_marker.c_str(),
           params.op.list_max_count.c_str());
@@ -938,7 +936,7 @@ class ListUserBucketsOp: virtual public DBOp {
 
 class PutObjectOp: virtual public DBOp {
   private:
-    const string Query =
+    const std::string Query =
       "INSERT OR REPLACE INTO '{}' \
       (ObjName, ObjInstance, ObjNS, BucketName, ACLs, IndexVer, Tag, \
        Flags, VersionedEpoch, ObjCategory, Etag, Owner, OwnerDisplayName, \
@@ -956,7 +954,7 @@ class PutObjectOp: virtual public DBOp {
   public:
     virtual ~PutObjectOp() {}
 
-    string Schema(DBOpPrepareParams &params) {
+    std::string Schema(DBOpPrepareParams &params) {
       return fmt::format(Query.c_str(),
           params.object_table.c_str(), params.op.obj.obj_name,
           params.op.obj.obj_instance, params.op.obj.obj_ns,
@@ -987,13 +985,13 @@ class PutObjectOp: virtual public DBOp {
 
 class DeleteObjectOp: virtual public DBOp {
   private:
-    const string Query =
+    const std::string Query =
       "DELETE from '{}' where BucketName = {} and ObjName = {} and ObjInstance = {}";
 
   public:
     virtual ~DeleteObjectOp() {}
 
-    string Schema(DBOpPrepareParams &params) {
+    std::string Schema(DBOpPrepareParams &params) {
       return fmt::format(Query.c_str(), params.object_table.c_str(),
           params.op.bucket.bucket_name.c_str(),
           params.op.obj.obj_name.c_str(),
@@ -1003,7 +1001,7 @@ class DeleteObjectOp: virtual public DBOp {
 
 class GetObjectOp: virtual public DBOp {
   private:
-    const string Query =
+    const std::string Query =
       "SELECT  \
       ObjName, ObjInstance, ObjNS, BucketName, ACLs, IndexVer, Tag, \
       Flags, VersionedEpoch, ObjCategory, Etag, Owner, OwnerDisplayName, \
@@ -1019,7 +1017,7 @@ class GetObjectOp: virtual public DBOp {
   public:
     virtual ~GetObjectOp() {}
 
-    string Schema(DBOpPrepareParams &params) {
+    std::string Schema(DBOpPrepareParams &params) {
       return fmt::format(Query.c_str(),
           params.object_table.c_str(),
           params.op.bucket.bucket_name.c_str(),
@@ -1032,7 +1030,7 @@ class ListBucketObjectsOp: virtual public DBOp {
   private:
     // once we have stats also stored, may have to update this query to join
     // these two tables.
-    const string Query =
+    const std::string Query =
       "SELECT  \
       ObjName, ObjInstance, ObjNS, BucketName, ACLs, IndexVer, Tag, \
       Flags, VersionedEpoch, ObjCategory, Etag, Owner, OwnerDisplayName, \
@@ -1047,7 +1045,7 @@ class ListBucketObjectsOp: virtual public DBOp {
   public:
     virtual ~ListBucketObjectsOp() {}
 
-    string Schema(DBOpPrepareParams &params) {
+    std::string Schema(DBOpPrepareParams &params) {
       /* XXX: Include prefix, delim */
       return fmt::format(Query.c_str(),
           params.object_table.c_str(),
@@ -1060,16 +1058,16 @@ class ListBucketObjectsOp: virtual public DBOp {
 class UpdateObjectOp: virtual public DBOp {
   private:
     // Updates Omap
-    const string OmapQuery =
+    const std::string OmapQuery =
       "UPDATE '{}' SET Omap = {}, Mtime = {} \
       where BucketName = {} and ObjName = {} and ObjInstance = {}";
-    const string AttrsQuery =
+    const std::string AttrsQuery =
       "UPDATE '{}' SET ObjAttrs = {}, Mtime = {}  \
       where BucketName = {} and ObjName = {} and ObjInstance = {}";
-    const string MPQuery =
+    const std::string MPQuery =
       "UPDATE '{}' SET MPPartsList = {}, Mtime = {}  \
       where BucketName = {} and ObjName = {} and ObjInstance = {}";
-    const string MetaQuery =
+    const std::string MetaQuery =
       "UPDATE '{}' SET \
        ObjNS = {}, ACLs = {}, IndexVer = {}, Tag = {}, Flags = {}, VersionedEpoch = {}, \
        ObjCategory = {}, Etag = {}, Owner = {}, OwnerDisplayName = {}, \
@@ -1088,7 +1086,7 @@ class UpdateObjectOp: virtual public DBOp {
   public:
     virtual ~UpdateObjectOp() {}
 
-    string Schema(DBOpPrepareParams &params) {
+    std::string Schema(DBOpPrepareParams &params) {
       if (params.op.query_str == "omap") {
         return fmt::format(OmapQuery.c_str(),
             params.object_table.c_str(), params.op.obj.omap.c_str(),
@@ -1147,7 +1145,7 @@ class UpdateObjectOp: virtual public DBOp {
 
 class PutObjectDataOp: virtual public DBOp {
   private:
-    const string Query =
+    const std::string Query =
       "INSERT OR REPLACE INTO '{}' \
       (ObjName, ObjInstance, ObjNS, BucketName, MultipartPartStr, PartNum, Offset, Size, Data) \
       VALUES ({}, {}, {}, {}, {}, {}, {}, {}, {})";
@@ -1155,7 +1153,7 @@ class PutObjectDataOp: virtual public DBOp {
   public:
     virtual ~PutObjectDataOp() {}
 
-    string Schema(DBOpPrepareParams &params) {
+    std::string Schema(DBOpPrepareParams &params) {
       return fmt::format(Query.c_str(),
           params.objectdata_table.c_str(),
           params.op.obj.obj_name, params.op.obj.obj_instance,
@@ -1171,7 +1169,7 @@ class PutObjectDataOp: virtual public DBOp {
 
 class UpdateObjectDataOp: virtual public DBOp {
   private:
-    const string Query =
+    const std::string Query =
       "UPDATE '{}' \
       SET ObjName = {}, ObjInstance = {}, ObjNS = {} \
       WHERE ObjName = {} and ObjInstance = {} and ObjNS = {} and \
@@ -1180,7 +1178,7 @@ class UpdateObjectDataOp: virtual public DBOp {
   public:
     virtual ~UpdateObjectDataOp() {}
 
-    string Schema(DBOpPrepareParams &params) {
+    std::string Schema(DBOpPrepareParams &params) {
       return fmt::format(Query.c_str(),
           params.objectdata_table.c_str(),
           params.op.obj.new_obj_name, params.op.obj.new_obj_instance,
@@ -1192,7 +1190,7 @@ class UpdateObjectDataOp: virtual public DBOp {
 };
 class GetObjectDataOp: virtual public DBOp {
   private:
-    const string Query =
+    const std::string Query =
       "SELECT  \
       ObjName, ObjInstance, ObjNS, BucketName, MultipartPartStr, PartNum, Offset, Size, Data \
       from '{}' where BucketName = {} and ObjName = {} and ObjInstance = {} ORDER BY MultipartPartStr, PartNum";
@@ -1200,7 +1198,7 @@ class GetObjectDataOp: virtual public DBOp {
   public:
     virtual ~GetObjectDataOp() {}
 
-    string Schema(DBOpPrepareParams &params) {
+    std::string Schema(DBOpPrepareParams &params) {
       return fmt::format(Query.c_str(),
           params.objectdata_table.c_str(),
           params.op.bucket.bucket_name.c_str(),
@@ -1211,13 +1209,13 @@ class GetObjectDataOp: virtual public DBOp {
 
 class DeleteObjectDataOp: virtual public DBOp {
   private:
-    const string Query =
+    const std::string Query =
       "DELETE from '{}' where BucketName = {} and ObjName = {} and ObjInstance = {}";
 
   public:
     virtual ~DeleteObjectDataOp() {}
 
-    string Schema(DBOpPrepareParams &params) {
+    std::string Schema(DBOpPrepareParams &params) {
       return fmt::format(Query.c_str(),
           params.objectdata_table.c_str(),
           params.op.bucket.bucket_name.c_str(),
@@ -1228,7 +1226,7 @@ class DeleteObjectDataOp: virtual public DBOp {
 
 class InsertLCEntryOp: virtual public DBOp {
   private:
-    const string Query =
+    const std::string Query =
       "INSERT OR REPLACE INTO '{}' \
       (LCIndex, BucketName, StartTime, Status) \
       VALUES ({}, {}, {}, {})";
@@ -1236,7 +1234,7 @@ class InsertLCEntryOp: virtual public DBOp {
   public:
     virtual ~InsertLCEntryOp() {}
 
-    string Schema(DBOpPrepareParams &params) {
+    std::string Schema(DBOpPrepareParams &params) {
       return fmt::format(Query.c_str(), params.lc_entry_table.c_str(),
           params.op.lc_entry.index, params.op.lc_entry.bucket_name,
           params.op.lc_entry.start_time, params.op.lc_entry.status);
@@ -1245,13 +1243,13 @@ class InsertLCEntryOp: virtual public DBOp {
 
 class RemoveLCEntryOp: virtual public DBOp {
   private:
-    const string Query =
+    const std::string Query =
       "DELETE from '{}' where LCIndex = {} and BucketName = {}";
 
   public:
     virtual ~RemoveLCEntryOp() {}
 
-    string Schema(DBOpPrepareParams &params) {
+    std::string Schema(DBOpPrepareParams &params) {
       return fmt::format(Query.c_str(), params.lc_entry_table.c_str(),
           params.op.lc_entry.index, params.op.lc_entry.bucket_name);
     }
@@ -1259,17 +1257,17 @@ class RemoveLCEntryOp: virtual public DBOp {
 
 class GetLCEntryOp: virtual public DBOp {
   private:
-    const string Query = "SELECT  \
+    const std::string Query = "SELECT  \
                           LCIndex, BucketName, StartTime, Status \
                           from '{}' where LCIndex = {} and BucketName = {}";
-    const string NextQuery = "SELECT  \
+    const std::string NextQuery = "SELECT  \
                           LCIndex, BucketName, StartTime, Status \
                           from '{}' where LCIndex = {} and BucketName > {} ORDER BY BucketName ASC";
 
   public:
     virtual ~GetLCEntryOp() {}
 
-    string Schema(DBOpPrepareParams &params) {
+    std::string Schema(DBOpPrepareParams &params) {
       if (params.op.query_str == "get_next_entry") {
         return fmt::format(NextQuery.c_str(), params.lc_entry_table.c_str(),
             params.op.lc_entry.index, params.op.lc_entry.bucket_name);
@@ -1282,14 +1280,14 @@ class GetLCEntryOp: virtual public DBOp {
 
 class ListLCEntriesOp: virtual public DBOp {
   private:
-    const string Query = "SELECT  \
+    const std::string Query = "SELECT  \
                           LCIndex, BucketName, StartTime, Status \
                           FROM '{}' WHERE LCIndex = {} AND BucketName > {} ORDER BY BucketName ASC LIMIT {}";
 
   public:
     virtual ~ListLCEntriesOp() {}
 
-    string Schema(DBOpPrepareParams &params) {
+    std::string Schema(DBOpPrepareParams &params) {
       return fmt::format(Query.c_str(), params.lc_entry_table.c_str(),
           params.op.lc_entry.index.c_str(), params.op.lc_entry.min_marker.c_str(),
           params.op.list_max_count.c_str());
@@ -1298,7 +1296,7 @@ class ListLCEntriesOp: virtual public DBOp {
 
 class InsertLCHeadOp: virtual public DBOp {
   private:
-    const string Query =
+    const std::string Query =
       "INSERT OR REPLACE INTO '{}' \
       (LCIndex, Marker, StartDate) \
       VALUES ({}, {}, {})";
@@ -1306,7 +1304,7 @@ class InsertLCHeadOp: virtual public DBOp {
   public:
     virtual ~InsertLCHeadOp() {}
 
-    string Schema(DBOpPrepareParams &params) {
+    std::string Schema(DBOpPrepareParams &params) {
       return fmt::format(Query.c_str(), params.lc_head_table.c_str(),
           params.op.lc_head.index, params.op.lc_head.marker,
           params.op.lc_head.start_date);
@@ -1315,13 +1313,13 @@ class InsertLCHeadOp: virtual public DBOp {
 
 class RemoveLCHeadOp: virtual public DBOp {
   private:
-    const string Query =
+    const std::string Query =
       "DELETE from '{}' where LCIndex = {}";
 
   public:
     virtual ~RemoveLCHeadOp() {}
 
-    string Schema(DBOpPrepareParams &params) {
+    std::string Schema(DBOpPrepareParams &params) {
       return fmt::format(Query.c_str(), params.lc_head_table.c_str(),
           params.op.lc_head.index);
     }
@@ -1329,14 +1327,14 @@ class RemoveLCHeadOp: virtual public DBOp {
 
 class GetLCHeadOp: virtual public DBOp {
   private:
-    const string Query = "SELECT  \
+    const std::string Query = "SELECT  \
                           LCIndex, Marker, StartDate \
                           from '{}' where LCIndex = {}";
 
   public:
     virtual ~GetLCHeadOp() {}
 
-    string Schema(DBOpPrepareParams &params) {
+    std::string Schema(DBOpPrepareParams &params) {
       return fmt::format(Query.c_str(), params.lc_head_table.c_str(),
           params.op.lc_head.index);
     }
@@ -1365,14 +1363,14 @@ WRITE_CLASS_ENCODER(DBOLHInfo)
 
 class DB {
   private:
-    const string db_name;
+    const std::string db_name;
     rgw::sal::Store* store;
-    const string user_table;
-    const string bucket_table;
-    const string quota_table;
-    const string lc_head_table;
-    const string lc_entry_table;
-    static map<string, class ObjectOp*> objectmap;
+    const std::string user_table;
+    const std::string bucket_table;
+    const std::string quota_table;
+    const std::string lc_head_table;
+    const std::string lc_entry_table;
+    static std::map<std::string, class ObjectOp*> objectmap;
 
   protected:
     void *db;
@@ -1386,8 +1384,8 @@ class DB {
     // objects if any.
     std::mutex mtx;
 
-  public:      
-    DB(string db_name, CephContext *_cct) : db_name(db_name),
+  public:
+    DB(std::string db_name, CephContext *_cct) : db_name(db_name),
     user_table(db_name+".user.table"),
     bucket_table(db_name+".bucket.table"),
     quota_table(db_name+".quota.table"),
@@ -1409,19 +1407,19 @@ class DB {
   {}
     virtual    ~DB() {}
 
-    const string getDBname() { return db_name; }
-    const string getDBfile() { return db_name + ".db"; }
-    const string getUserTable() { return user_table; }
-    const string getBucketTable() { return bucket_table; }
-    const string getQuotaTable() { return quota_table; }
-    const string getLCHeadTable() { return lc_head_table; }
-    const string getLCEntryTable() { return lc_entry_table; }
-    const string getObjectTable(string bucket) {
+    const std::string getDBname() { return db_name; }
+    const std::string getDBfile() { return db_name + ".db"; }
+    const std::string getUserTable() { return user_table; }
+    const std::string getBucketTable() { return bucket_table; }
+    const std::string getQuotaTable() { return quota_table; }
+    const std::string getLCHeadTable() { return lc_head_table; }
+    const std::string getLCEntryTable() { return lc_entry_table; }
+    const std::string getObjectTable(std::string bucket) {
       return db_name+"."+bucket+".object.table"; }
-    const string getObjectDataTable(string bucket) {
+    const std::string getObjectDataTable(std::string bucket) {
       return db_name+"."+bucket+".objectdata.table"; }
 
-    map<string, class ObjectOp*> getObjectMap();
+    std::map<std::string, class ObjectOp*> getObjectMap();
 
     struct DBOps dbops; // DB operations, make it private?
 
@@ -1436,18 +1434,18 @@ class DB {
     CephContext *ctx() { return cct; }
     const DoutPrefixProvider *get_def_dpp() { return &dp; }
 
-    int Initialize(string logfile, int loglevel);
+    int Initialize(std::string logfile, int loglevel);
     int Destroy(const DoutPrefixProvider *dpp);
     int LockInit(const DoutPrefixProvider *dpp);
     int LockDestroy(const DoutPrefixProvider *dpp);
     int Lock(const DoutPrefixProvider *dpp);
     int Unlock(const DoutPrefixProvider *dpp);
 
-    int InitializeParams(const DoutPrefixProvider *dpp, string Op, DBOpParams *params);
-    int ProcessOp(const DoutPrefixProvider *dpp, string Op, DBOpParams *params);
-    DBOp* getDBOp(const DoutPrefixProvider *dpp, string Op, struct DBOpParams *params);
-    int objectmapInsert(const DoutPrefixProvider *dpp, string bucket, class ObjectOp* ptr);
-    int objectmapDelete(const DoutPrefixProvider *dpp, string bucket);
+    int InitializeParams(const DoutPrefixProvider *dpp, std::string Op, DBOpParams *params);
+    int ProcessOp(const DoutPrefixProvider *dpp, std::string Op, DBOpParams *params);
+    DBOp* getDBOp(const DoutPrefixProvider *dpp, std::string Op, struct DBOpParams *params);
+    int objectmapInsert(const DoutPrefixProvider *dpp, std::string bucket, class ObjectOp* ptr);
+    int objectmapDelete(const DoutPrefixProvider *dpp, std::string bucket);
 
     virtual uint64_t get_blob_limit() { return 0; };
     virtual void *openDB(const DoutPrefixProvider *dpp) { return NULL; }
@@ -1464,10 +1462,10 @@ class DB {
 
     int get_user(const DoutPrefixProvider *dpp,
         const std::string& query_str, const std::string& query_str_val,
-        RGWUserInfo& uinfo, map<string, bufferlist> *pattrs,
+        RGWUserInfo& uinfo, std::map<std::string, bufferlist> *pattrs,
         RGWObjVersionTracker *pobjv_tracker);
     int store_user(const DoutPrefixProvider *dpp,
-        RGWUserInfo& uinfo, bool exclusive, map<string, bufferlist> *pattrs,
+        RGWUserInfo& uinfo, bool exclusive, std::map<std::string, bufferlist> *pattrs,
         RGWObjVersionTracker *pobjv_tracker, RGWUserInfo* pold_info);
     int remove_user(const DoutPrefixProvider *dpp,
         RGWUserInfo& uinfo, RGWObjVersionTracker *pobjv_tracker);
@@ -1477,11 +1475,11 @@ class DB {
         obj_version* pbucket_version);
     int create_bucket(const DoutPrefixProvider *dpp,
         const RGWUserInfo& owner, rgw_bucket& bucket,
-        const string& zonegroup_id,
+        const std::string& zonegroup_id,
         const rgw_placement_rule& placement_rule,
-        const string& swift_ver_location,
+        const std::string& swift_ver_location,
         const RGWQuotaInfo * pquota_info,
-        map<std::string, bufferlist>& attrs,
+        std::map<std::string, bufferlist>& attrs,
         RGWBucketInfo& info,
         obj_version *pobjv,
         obj_version *pep_objv,
@@ -1495,15 +1493,15 @@ class DB {
 
     int remove_bucket(const DoutPrefixProvider *dpp, const RGWBucketInfo info);
     int list_buckets(const DoutPrefixProvider *dpp, const rgw_user& user,
-        const string& marker,
-        const string& end_marker,
+        const std::string& marker,
+        const std::string& end_marker,
         uint64_t max,
         bool need_stats,
         RGWUserBuckets *buckets,
         bool *is_truncated);
     int update_bucket(const DoutPrefixProvider *dpp, const std::string& query_str,
         RGWBucketInfo& info, bool exclusive,
-        const rgw_user* powner_id, map<std::string, bufferlist>* pattrs,
+        const rgw_user* powner_id, std::map<std::string, bufferlist>* pattrs,
         ceph::real_time* pmtime, RGWObjVersionTracker* pobjv);
 
     uint64_t get_max_head_size() { return ObjHeadSize; }
@@ -1512,17 +1510,17 @@ class DB {
 
     // db raw obj string is of format -
     // "<bucketname>_<objname>_<objinstance>_<multipart-part-str>_<partnum>"
-    const string raw_obj_oid = "{0}_{1}_{2}_{3}_{4}";
+    const std::string raw_obj_oid = "{0}_{1}_{2}_{3}_{4}";
 
-    inline string to_oid(const string& bucket, const string& obj_name, const string& obj_instance,
-        string mp_str, uint64_t partnum) {
-      string s = fmt::format(raw_obj_oid.c_str(), bucket, obj_name, obj_instance, mp_str, partnum);
+    inline std::string to_oid(const std::string& bucket, const std::string& obj_name, const std::string& obj_instance,
+        std::string mp_str, uint64_t partnum) {
+      std::string s = fmt::format(raw_obj_oid.c_str(), bucket, obj_name, obj_instance, mp_str, partnum);
       return s;
     }
-    inline int from_oid(const string& oid, string& bucket, string& obj_name,
-        string& obj_instance,
-        string& mp_str, uint64_t& partnum) {
-      vector<std::string> result;
+    inline int from_oid(const std::string& oid, std::string& bucket, std::string& obj_name,
+        std::string& obj_instance,
+        std::string& mp_str, uint64_t& partnum) {
+      std::vector<std::string> result;
       boost::split(result, oid, boost::is_any_of("_"));
       bucket = result[0];
       obj_name = result[1];
@@ -1536,22 +1534,22 @@ class DB {
     struct raw_obj {
       DB* db;
 
-      string bucket_name;
-      string obj_name;
-      string obj_instance;
-      string obj_ns;
-      string multipart_part_str;
+      std::string bucket_name;
+      std::string obj_name;
+      std::string obj_instance;
+      std::string obj_ns;
+      std::string multipart_part_str;
       uint64_t part_num;
 
-      string obj_table;
-      string obj_data_table;
+      std::string obj_table;
+      std::string obj_data_table;
 
       raw_obj(DB* _db) {
         db = _db;
       }
 
-      raw_obj(DB* _db, string& _bname, string& _obj_name, string& _obj_instance,
-          string& _obj_ns, string _mp_part_str, int _part_num) {
+      raw_obj(DB* _db, std::string& _bname, std::string& _obj_name, std::string& _obj_instance,
+          std::string& _obj_ns, std::string _mp_part_str, int _part_num) {
         db = _db;
         bucket_name = _bname;
         obj_name = _obj_name;
@@ -1564,7 +1562,7 @@ class DB {
         obj_data_table = bucket_name+".objectdata.table";
       }
 
-      raw_obj(DB* _db, string& oid) {
+      raw_obj(DB* _db, std::string& oid) {
         int r;
 
         db = _db;
@@ -1609,11 +1607,11 @@ class DB {
       public:
 
         struct Params {
-          string prefix;
-          string delim;
+          std::string prefix;
+          std::string delim;
           rgw_obj_key marker;
           rgw_obj_key end_marker;
-          string ns;
+          std::string ns;
           bool enforce_ns;
           RGWAccessListFilter* access_list_filter;
           RGWBucketListNameFilter force_check_filter;
@@ -1633,8 +1631,8 @@ class DB {
         /* XXX: Handle ordered and unordered separately.
          * For now returning only ordered entries */
         int list_objects(const DoutPrefixProvider *dpp, int64_t max,
-                          vector<rgw_bucket_dir_entry> *result,
-                          map<string, bool> *common_prefixes, bool *is_truncated);
+                          std::vector<rgw_bucket_dir_entry> *result,
+                          std::map<std::string, bool> *common_prefixes, bool *is_truncated);
         rgw_obj_key& get_next_marker() {
           return next_marker;
         }
@@ -1678,7 +1676,7 @@ class DB {
           const char *if_match;
           const char *if_nomatch;
 
-          ConditionParams() : 
+          ConditionParams() :
             mod_ptr(NULL), unmod_ptr(NULL), high_precision_time(false), mod_zone_id(0), mod_pg_ver(0),
             if_match(NULL), if_nomatch(NULL) {}
         } conds;
@@ -1686,7 +1684,7 @@ class DB {
         struct Params {
           ceph::real_time *lastmod;
           uint64_t *obj_size;
-          map<string, bufferlist> *attrs;
+         std::map<std::string, bufferlist> *attrs;
           rgw_obj *target_obj;
 
           Params() : lastmod(nullptr), obj_size(nullptr), attrs(nullptr),
@@ -1705,15 +1703,15 @@ class DB {
       struct Write {
         DB::Object *target;
         RGWObjState obj_state;
-        string mp_part_str = "0.0"; // multipart num
+        std::string mp_part_str = "0.0"; // multipart num
 
         struct MetaParams {
           ceph::real_time *mtime;
-          map<std::string, bufferlist>* rmattrs;
+         std::map<std::string, bufferlist>* rmattrs;
           const bufferlist *data;
           RGWObjManifest *manifest;
-          const string *ptag;
-          list<rgw_obj_index_key> *remove_objs;
+          const std::string *ptag;
+          std::list<rgw_obj_index_key> *remove_objs;
           ceph::real_time set_mtime;
           rgw_user owner;
           RGWObjCategory category;
@@ -1723,7 +1721,7 @@ class DB {
           std::optional<uint64_t> olh_epoch;
           ceph::real_time delete_at;
           bool canceled;
-          const string *user_data;
+          const std::string *user_data;
           rgw_zone_set *zones_trace;
           bool modify_tail;
           bool completeMultipart;
@@ -1737,16 +1735,16 @@ class DB {
 
         explicit Write(DB::Object *_target) : target(_target) {}
 
-        void set_mp_part_str(string _mp_part_str) { mp_part_str = _mp_part_str;}
+        void set_mp_part_str(std::string _mp_part_str) { mp_part_str = _mp_part_str;}
         int prepare(const DoutPrefixProvider* dpp);
         int write_data(const DoutPrefixProvider* dpp,
                                bufferlist& data, uint64_t ofs);
         int _do_write_meta(const DoutPrefixProvider *dpp,
             uint64_t size, uint64_t accounted_size,
-            map<string, bufferlist>& attrs,
+           std::map<std::string, bufferlist>& attrs,
             bool assume_noent, bool modify_tail);
         int write_meta(const DoutPrefixProvider *dpp, uint64_t size,
-            uint64_t accounted_size, map<string, bufferlist>& attrs);
+           uint64_t accounted_size, std::map<std::string, bufferlist>& attrs);
         /* Below are used to update mp data rows object name
          * from meta to src object name on multipart upload
          * completion
@@ -1762,9 +1760,9 @@ class DB {
           int versioning_status;
           ACLOwner obj_owner; /* needed for creation of deletion marker */
           uint64_t olh_epoch;
-          string marker_version_id;
+          std::string marker_version_id;
           uint32_t bilog_flags;
-          list<rgw_obj_index_key> *remove_objs;
+          std::list<rgw_obj_index_key> *remove_objs;
           ceph::real_time expiration_time;
           ceph::real_time unmod_since;
           ceph::real_time mtime; /* for setting delete marker mtime */
@@ -1778,7 +1776,7 @@ class DB {
 
         struct DeleteResult {
           bool delete_marker;
-          string version_id;
+          std::string version_id;
 
           DeleteResult() : delete_marker(false) {}
         } result;
@@ -1806,8 +1804,8 @@ class DB {
       RGWBucketInfo& get_bucket_info() { return bucket_info; }
 
       int InitializeParamsfromObject(const DoutPrefixProvider *dpp, DBOpParams* params);
-      int set_attrs(const DoutPrefixProvider *dpp, map<string, bufferlist>& setattrs,
-          map<string, bufferlist>* rmattrs);
+      int set_attrs(const DoutPrefixProvider *dpp, std::map<std::string, bufferlist>& setattrs,
+          std::map<std::string, bufferlist>* rmattrs);
       int obj_omap_set_val_by_key(const DoutPrefixProvider *dpp, const std::string& key, bufferlist& val, bool must_exist);
       int obj_omap_get_vals_by_keys(const DoutPrefixProvider *dpp, const std::string& oid,
           const std::set<std::string>& keys,
index 9bd4a2274802b6ff744784817f12d9e766e2c129..4b1373bfccb2664d4d8c0464d8179e811f02a2bb 100644 (file)
@@ -4,15 +4,13 @@
 #ifndef DB_STORE_LOG_H
 #define DB_STORE_LOG_H
 
-#include <errno.h>
-#include <stdlib.h>
+#include <cerrno>
+#include <cstdlib>
 #include <string>
-#include <stdio.h>
+#include <cstdio>
 #include <iostream>
 #include <fstream>
-#include "common/dout.h" 
-
-using namespace std;
+#include "common/dout.h"
 
 #define dout_subsys ceph_subsys_rgw
 #undef dout_prefix
index 0055191908b8e688075f2265c415d7f32fbfe5c2..34762b52d90cbdb7dcd0b64cd20fc2c4e2411b01 100644 (file)
@@ -8,6 +8,7 @@
 #include <dbstore.h>
 #include <dbstore_log.h>
 
+using namespace std;
 using namespace rgw::store;
 using DB = rgw::store::DB;
 
index 61e11d996370a205ac961db7252962e4f1009603..9da98a487dcdda921f085d6b4dcafd0c4b52717a 100644 (file)
@@ -4,6 +4,8 @@
 #include "dbstore_mgr.h"
 #include "common/dbstore_log.h"
 
+using namespace std;
+
 /* Given a tenant, find and return the DBStore handle.
  * If not found and 'create' set to true, create one
  * and return
index d3053e6ac7cb2301769ca119fc5ec81d6d5b39ab..850f97a42e27b05ffa982d5c7c1a9f0c4b6250cb 100644 (file)
@@ -4,27 +4,26 @@
 #pragma once
 
 #include <map>
-#include <errno.h>
-#include <stdlib.h>
+#include <cerrno>
+#include <cstdlib>
 #include <string>
-#include <stdio.h>
+#include <cstdio>
 #include <iostream>
+#include <vector>
+
 #include "common/ceph_context.h"
 #include "common/dbstore.h"
 #include "sqlite/sqliteDB.h"
 
-using namespace std;
 using namespace rgw::store;
 using DB = rgw::store::DB;
 
 /* XXX: Should be a dbstore config option */
-const static string default_tenant = "default_ns";
-
-using namespace std;
+const static std::string default_tenant = "default_ns";
 
 class DBStoreManager {
 private:
-  map<string, DB*> DBStoreHandles;
+  std::map<std::string, DB*> DBStoreHandles;
   DB *default_db = NULL;
   CephContext *cct;
 
@@ -33,9 +32,9 @@ public:
     cct = _cct;
        default_db = createDB(default_tenant);
   };
-  DBStoreManager(string logfile, int loglevel): DBStoreHandles() {
+  DBStoreManager(std::string logfile, int loglevel): DBStoreHandles() {
     /* No ceph context. Create one with log args provided */
-    vector<const char*> args;
+    std::vector<const char*> args;
     cct = global_init(NULL, args, CEPH_ENTITY_TYPE_CLIENT,
                       CODE_ENVIRONMENT_DAEMON, CINIT_FLAG_NO_MON_CONFIG, 1)->get();
     cct->_log->set_log_file(logfile);
@@ -50,9 +49,9 @@ public:
    * being deleted while using it.
    */
   DB* getDB () { return default_db; };
-  DB* getDB (string tenant, bool create);
-  DB* createDB (string tenant);
-  void deleteDB (string tenant);
+  DB* getDB (std::string tenant, bool create);
+  DB* createDB (std::string tenant);
+  void deleteDB (std::string tenant);
   void deleteDB (DB* db);
   void destroyAllHandles();
 };
index f24eb68b2e25a4b1496d7a7b2612ca8c73668604..6675af2922ec6b3667226a47c285917db496e068 100644 (file)
@@ -3,6 +3,8 @@
 
 #include "sqliteDB.h"
 
+using namespace std;
+
 #define SQL_PREPARE(dpp, params, sdb, stmt, ret, Op)   \
   do {                                                 \
     string schema;                                     \
index 5bf7dfdd16f7875111c6bbf0e7496b84b65163d7..dd56ab1f9ec669988faee0ec1166d9318c5765f6 100644 (file)
@@ -10,7 +10,6 @@
 #include <sqlite3.h>
 #include "rgw/store/dbstore/common/dbstore.h"
 
-using namespace std;
 using namespace rgw::store;
 
 class SQLiteDB : public DB, virtual public DBOp {
@@ -20,15 +19,15 @@ class SQLiteDB : public DB, virtual public DBOp {
   protected:
     CephContext *cct;
 
-  public:      
+  public:
     sqlite3_stmt *stmt = NULL;
     DBOpPrepareParams PrepareParams;
 
-    SQLiteDB(sqlite3 *dbi, string db_name, CephContext *_cct) : DB(db_name, _cct), cct(_cct) {
+    SQLiteDB(sqlite3 *dbi, std::string db_name, CephContext *_cct) : DB(db_name, _cct), cct(_cct) {
       db = (void*)dbi;
       InitPrepareParams(get_def_dpp(), PrepareParams);
     }
-    SQLiteDB(string db_name, CephContext *_cct) : DB(db_name, _cct), cct(_cct) {
+    SQLiteDB(std::string db_name, CephContext *_cct) : DB(db_name, _cct), cct(_cct) {
       InitPrepareParams(get_def_dpp(), PrepareParams);
     }
     ~SQLiteDB() {}
@@ -48,7 +47,7 @@ class SQLiteDB : public DB, virtual public DBOp {
     int Reset(const DoutPrefixProvider *dpp, sqlite3_stmt *stmt);
     /* default value matches with sqliteDB style */
 
-    int createTables(const DoutPrefixProvider *dpp);
+    int createTables(const DoutPrefixProvider *dpp) override;
     int createBucketTable(const DoutPrefixProvider *dpp, DBOpParams *params);
     int createUserTable(const DoutPrefixProvider *dpp, DBOpParams *params);
     int createObjectTable(const DoutPrefixProvider *dpp, DBOpParams *params);
@@ -79,7 +78,7 @@ class SQLObjectOp : public ObjectOp {
     SQLObjectOp(sqlite3 **sdbi, CephContext *_cct) : sdb(sdbi), cct(_cct) {};
     ~SQLObjectOp() {}
 
-    int InitializeObjectOps(string db_name, const DoutPrefixProvider *dpp);
+    int InitializeObjectOps(std::string db_name, const DoutPrefixProvider *dpp);
     int FreeObjectOps(const DoutPrefixProvider *dpp);
 };
 
@@ -89,7 +88,7 @@ class SQLInsertUser : public SQLiteDB, public InsertUserOp {
     sqlite3_stmt *stmt = NULL; // Prepared statement
 
   public:
-    SQLInsertUser(void **db, string db_name, CephContext *cct) : SQLiteDB((sqlite3 *)(*db), db_name, cct), sdb((sqlite3 **)db) {}
+    SQLInsertUser(void **db, std::string db_name, CephContext *cct) : SQLiteDB((sqlite3 *)(*db), db_name, cct), sdb((sqlite3 **)db) {}
     ~SQLInsertUser() {
       if (stmt)
         sqlite3_finalize(stmt);
@@ -105,7 +104,7 @@ class SQLRemoveUser : public SQLiteDB, public RemoveUserOp {
     sqlite3_stmt *stmt = NULL; // Prepared statement
 
   public:
-    SQLRemoveUser(void **db, string db_name, CephContext *cct) : SQLiteDB((sqlite3 *)(*db), db_name, cct), sdb((sqlite3 **)db) {}
+    SQLRemoveUser(void **db, std::string db_name, CephContext *cct) : SQLiteDB((sqlite3 *)(*db), db_name, cct), sdb((sqlite3 **)db) {}
     ~SQLRemoveUser() {
       if (stmt)
         sqlite3_finalize(stmt);
@@ -124,7 +123,7 @@ class SQLGetUser : public SQLiteDB, public GetUserOp {
     sqlite3_stmt *userid_stmt = NULL; // Prepared statement to query by user_id
 
   public:
-    SQLGetUser(void **db, string db_name, CephContext *cct) : SQLiteDB((sqlite3 *)(*db), db_name, cct), sdb((sqlite3 **)db) {}
+    SQLGetUser(void **db, std::string db_name, CephContext *cct) : SQLiteDB((sqlite3 *)(*db), db_name, cct), sdb((sqlite3 **)db) {}
     ~SQLGetUser() {
       if (stmt)
         sqlite3_finalize(stmt);
@@ -146,7 +145,7 @@ class SQLInsertBucket : public SQLiteDB, public InsertBucketOp {
     sqlite3_stmt *stmt = NULL; // Prepared statement
 
   public:
-    SQLInsertBucket(void **db, string db_name, CephContext *cct) : SQLiteDB((sqlite3 *)(*db), db_name, cct), sdb((sqlite3 **)db) {}
+    SQLInsertBucket(void **db, std::string db_name, CephContext *cct) : SQLiteDB((sqlite3 *)(*db), db_name, cct), sdb((sqlite3 **)db) {}
     ~SQLInsertBucket() {
       if (stmt)
         sqlite3_finalize(stmt);
@@ -164,7 +163,7 @@ class SQLUpdateBucket : public SQLiteDB, public UpdateBucketOp {
     sqlite3_stmt *owner_stmt = NULL; // Prepared statement
 
   public:
-    SQLUpdateBucket(void **db, string db_name, CephContext *cct) : SQLiteDB((sqlite3 *)(*db), db_name, cct), sdb((sqlite3 **)db) {}
+    SQLUpdateBucket(void **db, std::string db_name, CephContext *cct) : SQLiteDB((sqlite3 *)(*db), db_name, cct), sdb((sqlite3 **)db) {}
     ~SQLUpdateBucket() {
       if (info_stmt)
         sqlite3_finalize(info_stmt);
@@ -184,7 +183,7 @@ class SQLRemoveBucket : public SQLiteDB, public RemoveBucketOp {
     sqlite3_stmt *stmt = NULL; // Prepared statement
 
   public:
-    SQLRemoveBucket(void **db, string db_name, CephContext *cct) : SQLiteDB((sqlite3 *)(*db), db_name, cct), sdb((sqlite3 **)db) {}
+    SQLRemoveBucket(void **db, std::string db_name, CephContext *cct) : SQLiteDB((sqlite3 *)(*db), db_name, cct), sdb((sqlite3 **)db) {}
     ~SQLRemoveBucket() {
       if (stmt)
         sqlite3_finalize(stmt);
@@ -200,7 +199,7 @@ class SQLGetBucket : public SQLiteDB, public GetBucketOp {
     sqlite3_stmt *stmt = NULL; // Prepared statement
 
   public:
-    SQLGetBucket(void **db, string db_name, CephContext *cct) : SQLiteDB((sqlite3 *)(*db), db_name, cct), sdb((sqlite3 **)db) {}
+    SQLGetBucket(void **db, std::string db_name, CephContext *cct) : SQLiteDB((sqlite3 *)(*db), db_name, cct), sdb((sqlite3 **)db) {}
     ~SQLGetBucket() {
       if (stmt)
         sqlite3_finalize(stmt);
@@ -216,7 +215,7 @@ class SQLListUserBuckets : public SQLiteDB, public ListUserBucketsOp {
     sqlite3_stmt *stmt = NULL; // Prepared statement
 
   public:
-    SQLListUserBuckets(void **db, string db_name, CephContext *cct) : SQLiteDB((sqlite3 *)(*db), db_name, cct), sdb((sqlite3 **)db) {}
+    SQLListUserBuckets(void **db, std::string db_name, CephContext *cct) : SQLiteDB((sqlite3 *)(*db), db_name, cct), sdb((sqlite3 **)db) {}
     ~SQLListUserBuckets() {
       if (stmt)
         sqlite3_finalize(stmt);
@@ -232,8 +231,8 @@ class SQLPutObject : public SQLiteDB, public PutObjectOp {
     sqlite3_stmt *stmt = NULL; // Prepared statement
 
   public:
-    SQLPutObject(void **db, string db_name, CephContext *cct) : SQLiteDB((sqlite3 *)(*db), db_name, cct), sdb((sqlite3 **)db) {}
-    SQLPutObject(sqlite3 **sdbi, string db_name, CephContext *cct) : SQLiteDB(*sdbi, db_name, cct), sdb(sdbi) {}
+    SQLPutObject(void **db, std::string db_name, CephContext *cct) : SQLiteDB((sqlite3 *)(*db), db_name, cct), sdb((sqlite3 **)db) {}
+    SQLPutObject(sqlite3 **sdbi, std::string db_name, CephContext *cct) : SQLiteDB(*sdbi, db_name, cct), sdb(sdbi) {}
 
     ~SQLPutObject() {
       if (stmt)
@@ -250,8 +249,8 @@ class SQLDeleteObject : public SQLiteDB, public DeleteObjectOp {
     sqlite3_stmt *stmt = NULL; // Prepared statement
 
   public:
-    SQLDeleteObject(void **db, string db_name, CephContext *cct) : SQLiteDB((sqlite3 *)(*db), db_name, cct), sdb((sqlite3 **)db) {}
-    SQLDeleteObject(sqlite3 **sdbi, string db_name, CephContext *cct) : SQLiteDB(*sdbi, db_name, cct), sdb(sdbi) {}
+    SQLDeleteObject(void **db, std::string db_name, CephContext *cct) : SQLiteDB((sqlite3 *)(*db), db_name, cct), sdb((sqlite3 **)db) {}
+    SQLDeleteObject(sqlite3 **sdbi, std::string db_name, CephContext *cct) : SQLiteDB(*sdbi, db_name, cct), sdb(sdbi) {}
 
     ~SQLDeleteObject() {
       if (stmt)
@@ -268,8 +267,8 @@ class SQLGetObject : public SQLiteDB, public GetObjectOp {
     sqlite3_stmt *stmt = NULL; // Prepared statement
 
   public:
-    SQLGetObject(void **db, string db_name, CephContext *cct) : SQLiteDB((sqlite3 *)(*db), db_name, cct), sdb((sqlite3 **)db) {}
-    SQLGetObject(sqlite3 **sdbi, string db_name, CephContext *cct) : SQLiteDB(*sdbi, db_name, cct), sdb(sdbi) {}
+    SQLGetObject(void **db, std::string db_name, CephContext *cct) : SQLiteDB((sqlite3 *)(*db), db_name, cct), sdb((sqlite3 **)db) {}
+    SQLGetObject(sqlite3 **sdbi, std::string db_name, CephContext *cct) : SQLiteDB(*sdbi, db_name, cct), sdb(sdbi) {}
 
     ~SQLGetObject() {
       if (stmt)
@@ -289,8 +288,8 @@ class SQLUpdateObject : public SQLiteDB, public UpdateObjectOp {
     sqlite3_stmt *mp_stmt = NULL; // Prepared statement
 
   public:
-    SQLUpdateObject(void **db, string db_name, CephContext *cct) : SQLiteDB((sqlite3 *)(*db), db_name, cct), sdb((sqlite3 **)db) {}
-    SQLUpdateObject(sqlite3 **sdbi, string db_name, CephContext *cct) : SQLiteDB(*sdbi, db_name, cct), sdb(sdbi) {}
+    SQLUpdateObject(void **db, std::string db_name, CephContext *cct) : SQLiteDB((sqlite3 *)(*db), db_name, cct), sdb((sqlite3 **)db) {}
+    SQLUpdateObject(sqlite3 **sdbi, std::string db_name, CephContext *cct) : SQLiteDB(*sdbi, db_name, cct), sdb(sdbi) {}
 
     ~SQLUpdateObject() {
       if (omap_stmt)
@@ -312,8 +311,8 @@ class SQLListBucketObjects : public SQLiteDB, public ListBucketObjectsOp {
     sqlite3_stmt *stmt = NULL; // Prepared statement
 
   public:
-    SQLListBucketObjects(void **db, string db_name, CephContext *cct) : SQLiteDB((sqlite3 *)(*db), db_name, cct), sdb((sqlite3 **)db) {}
-    SQLListBucketObjects(sqlite3 **sdbi, string db_name, CephContext *cct) : SQLiteDB(*sdbi, db_name, cct), sdb(sdbi) {}
+    SQLListBucketObjects(void **db, std::string db_name, CephContext *cct) : SQLiteDB((sqlite3 *)(*db), db_name, cct), sdb((sqlite3 **)db) {}
+    SQLListBucketObjects(sqlite3 **sdbi, std::string db_name, CephContext *cct) : SQLiteDB(*sdbi, db_name, cct), sdb(sdbi) {}
 
     ~SQLListBucketObjects() {
       if (stmt)
@@ -330,8 +329,8 @@ class SQLPutObjectData : public SQLiteDB, public PutObjectDataOp {
     sqlite3_stmt *stmt = NULL; // Prepared statement
 
   public:
-    SQLPutObjectData(void **db, string db_name, CephContext *cct) : SQLiteDB((sqlite3 *)(*db), db_name, cct), sdb((sqlite3 **)db) {}
-    SQLPutObjectData(sqlite3 **sdbi, string db_name, CephContext *cct) : SQLiteDB(*sdbi, db_name, cct), sdb(sdbi) {}
+    SQLPutObjectData(void **db, std::string db_name, CephContext *cct) : SQLiteDB((sqlite3 *)(*db), db_name, cct), sdb((sqlite3 **)db) {}
+    SQLPutObjectData(sqlite3 **sdbi, std::string db_name, CephContext *cct) : SQLiteDB(*sdbi, db_name, cct), sdb(sdbi) {}
 
     ~SQLPutObjectData() {
       if (stmt)
@@ -348,8 +347,8 @@ class SQLUpdateObjectData : public SQLiteDB, public UpdateObjectDataOp {
     sqlite3_stmt *stmt = NULL; // Prepared statement
 
   public:
-    SQLUpdateObjectData(void **db, string db_name, CephContext *cct) : SQLiteDB((sqlite3 *)(*db), db_name, cct), sdb((sqlite3 **)db) {}
-    SQLUpdateObjectData(sqlite3 **sdbi, string db_name, CephContext *cct) : SQLiteDB(*sdbi, db_name, cct), sdb(sdbi) {}
+    SQLUpdateObjectData(void **db, std::string db_name, CephContext *cct) : SQLiteDB((sqlite3 *)(*db), db_name, cct), sdb((sqlite3 **)db) {}
+    SQLUpdateObjectData(sqlite3 **sdbi, std::string db_name, CephContext *cct) : SQLiteDB(*sdbi, db_name, cct), sdb(sdbi) {}
 
     ~SQLUpdateObjectData() {
       if (stmt)
@@ -366,8 +365,8 @@ class SQLGetObjectData : public SQLiteDB, public GetObjectDataOp {
     sqlite3_stmt *stmt = NULL; // Prepared statement
 
   public:
-    SQLGetObjectData(void **db, string db_name, CephContext *cct) : SQLiteDB((sqlite3 *)(*db), db_name, cct), sdb((sqlite3 **)db) {}
-    SQLGetObjectData(sqlite3 **sdbi, string db_name, CephContext *cct) : SQLiteDB(*sdbi, db_name, cct), sdb(sdbi) {}
+    SQLGetObjectData(void **db, std::string db_name, CephContext *cct) : SQLiteDB((sqlite3 *)(*db), db_name, cct), sdb((sqlite3 **)db) {}
+    SQLGetObjectData(sqlite3 **sdbi, std::string db_name, CephContext *cct) : SQLiteDB(*sdbi, db_name, cct), sdb(sdbi) {}
 
     ~SQLGetObjectData() {
       if (stmt)
@@ -384,8 +383,8 @@ class SQLDeleteObjectData : public SQLiteDB, public DeleteObjectDataOp {
     sqlite3_stmt *stmt = NULL; // Prepared statement
 
   public:
-    SQLDeleteObjectData(void **db, string db_name, CephContext *cct) : SQLiteDB((sqlite3 *)(*db), db_name, cct), sdb((sqlite3 **)db) {}
-    SQLDeleteObjectData(sqlite3 **sdbi, string db_name, CephContext *cct) : SQLiteDB(*sdbi, db_name, cct), sdb(sdbi) {}
+    SQLDeleteObjectData(void **db, std::string db_name, CephContext *cct) : SQLiteDB((sqlite3 *)(*db), db_name, cct), sdb((sqlite3 **)db) {}
+    SQLDeleteObjectData(sqlite3 **sdbi, std::string db_name, CephContext *cct) : SQLiteDB(*sdbi, db_name, cct), sdb(sdbi) {}
 
     ~SQLDeleteObjectData() {
       if (stmt)
@@ -402,7 +401,7 @@ class SQLInsertLCEntry : public SQLiteDB, public InsertLCEntryOp {
     sqlite3_stmt *stmt = NULL; // Prepared statement
 
   public:
-    SQLInsertLCEntry(void **db, string db_name, CephContext *cct) : SQLiteDB((sqlite3 *)(*db), db_name, cct), sdb((sqlite3 **)db) {}
+    SQLInsertLCEntry(void **db, std::string db_name, CephContext *cct) : SQLiteDB((sqlite3 *)(*db), db_name, cct), sdb((sqlite3 **)db) {}
     ~SQLInsertLCEntry() {
       if (stmt)
         sqlite3_finalize(stmt);
@@ -418,7 +417,7 @@ class SQLRemoveLCEntry : public SQLiteDB, public RemoveLCEntryOp {
     sqlite3_stmt *stmt = NULL; // Prepared statement
 
   public:
-    SQLRemoveLCEntry(void **db, string db_name, CephContext *cct) : SQLiteDB((sqlite3 *)(*db), db_name, cct), sdb((sqlite3 **)db) {}
+    SQLRemoveLCEntry(void **db, std::string db_name, CephContext *cct) : SQLiteDB((sqlite3 *)(*db), db_name, cct), sdb((sqlite3 **)db) {}
     ~SQLRemoveLCEntry() {
       if (stmt)
         sqlite3_finalize(stmt);
@@ -435,7 +434,7 @@ class SQLGetLCEntry : public SQLiteDB, public GetLCEntryOp {
     sqlite3_stmt *next_stmt = NULL; // Prepared statement
 
   public:
-    SQLGetLCEntry(void **db, string db_name, CephContext *cct) : SQLiteDB((sqlite3 *)(*db), db_name, cct), sdb((sqlite3 **)db) {}
+    SQLGetLCEntry(void **db, std::string db_name, CephContext *cct) : SQLiteDB((sqlite3 *)(*db), db_name, cct), sdb((sqlite3 **)db) {}
     ~SQLGetLCEntry() {
       if (stmt)
         sqlite3_finalize(stmt);
@@ -453,7 +452,7 @@ class SQLListLCEntries : public SQLiteDB, public ListLCEntriesOp {
     sqlite3_stmt *stmt = NULL; // Prepared statement
 
   public:
-    SQLListLCEntries(void **db, string db_name, CephContext *cct) : SQLiteDB((sqlite3 *)(*db), db_name, cct), sdb((sqlite3 **)db) {}
+    SQLListLCEntries(void **db, std::string db_name, CephContext *cct) : SQLiteDB((sqlite3 *)(*db), db_name, cct), sdb((sqlite3 **)db) {}
     ~SQLListLCEntries() {
       if (stmt)
         sqlite3_finalize(stmt);
@@ -469,7 +468,7 @@ class SQLInsertLCHead : public SQLiteDB, public InsertLCHeadOp {
     sqlite3_stmt *stmt = NULL; // Prepared statement
 
   public:
-    SQLInsertLCHead(void **db, string db_name, CephContext *cct) : SQLiteDB((sqlite3 *)(*db), db_name, cct), sdb((sqlite3 **)db) {}
+    SQLInsertLCHead(void **db, std::string db_name, CephContext *cct) : SQLiteDB((sqlite3 *)(*db), db_name, cct), sdb((sqlite3 **)db) {}
     ~SQLInsertLCHead() {
       if (stmt)
         sqlite3_finalize(stmt);
@@ -485,7 +484,7 @@ class SQLRemoveLCHead : public SQLiteDB, public RemoveLCHeadOp {
     sqlite3_stmt *stmt = NULL; // Prepared statement
 
   public:
-    SQLRemoveLCHead(void **db, string db_name, CephContext *cct) : SQLiteDB((sqlite3 *)(*db), db_name, cct), sdb((sqlite3 **)db) {}
+    SQLRemoveLCHead(void **db, std::string db_name, CephContext *cct) : SQLiteDB((sqlite3 *)(*db), db_name, cct), sdb((sqlite3 **)db) {}
     ~SQLRemoveLCHead() {
       if (stmt)
         sqlite3_finalize(stmt);
@@ -501,7 +500,7 @@ class SQLGetLCHead : public SQLiteDB, public GetLCHeadOp {
     sqlite3_stmt *stmt = NULL; // Prepared statement
 
   public:
-    SQLGetLCHead(void **db, string db_name, CephContext *cct) : SQLiteDB((sqlite3 *)(*db), db_name, cct), sdb((sqlite3 **)db) {}
+    SQLGetLCHead(void **db, std::string db_name, CephContext *cct) : SQLiteDB((sqlite3 *)(*db), db_name, cct), sdb((sqlite3 **)db) {}
     ~SQLGetLCHead() {
       if (stmt)
         sqlite3_finalize(stmt);