diff options
author | Yaniv Kaul <ykaul@redhat.com> | 2018-07-16 17:03:17 +0300 |
---|---|---|
committer | Amar Tumballi <amarts@redhat.com> | 2018-07-22 14:40:16 +0000 |
commit | 621138ce763eda8270d0a4f6d7209fd50ada8787 (patch) | |
tree | 7299759993f6e9f7f34dad95fc8d8cd6ffc1b2fd /xlators/features/changetimerecorder | |
parent | c0e76377d0fc47aa66f35ea70a4793731ebbd40c (diff) |
All: run codespell on the code and fix issues.
Please review, it's not always just the comments that were fixed.
I've had to revert of course all calls to creat() that were changed
to create() ...
Only compile-tested!
Change-Id: I7d02e82d9766e272a7fd9cc68e51901d69e5aab5
updates: bz#1193929
Signed-off-by: Yaniv Kaul <ykaul@redhat.com>
Diffstat (limited to 'xlators/features/changetimerecorder')
3 files changed, 16 insertions, 16 deletions
diff --git a/xlators/features/changetimerecorder/src/changetimerecorder.c b/xlators/features/changetimerecorder/src/changetimerecorder.c index 40bfcf4a32a..a9d73ed0ac8 100644 --- a/xlators/features/changetimerecorder/src/changetimerecorder.c +++ b/xlators/features/changetimerecorder/src/changetimerecorder.c @@ -48,7 +48,7 @@ These are the scenarios when the hardlink list is updated: 1) Named-Lookup: Whenever a named lookup happens on a file, in the wind path we copy all required hardlink and inode information to ctr_db_record structure, which resides in the frame->local variable. - We dont update the database in wind. During the unwind, we read the + We don't update the database in wind. During the unwind, we read the information from the ctr_db_record and , Check if the inode context variable is created, if not we create it. Check if the hard link is there in the hardlink list. @@ -74,12 +74,12 @@ These are the scenarios when the hardlink list is updated: What is pending: 1) This solution will only work for named lookups. -2) We dont track afr-self-heal/dht-rebalancer traffic for healing. +2) We don't track afr-self-heal/dht-rebalancer traffic for healing. */ -/* This function doesnot write anything to the db, +/* This function does not write anything to the db, * just created the local variable * for the frame and sets values for the ctr_db_record */ static int @@ -109,9 +109,9 @@ ctr_lookup_wind(call_frame_t *frame, goto out; }; ctr_local = frame->local; - /*Definately no internal fops will reach here*/ + /*Definitely no internal fops will reach here*/ ctr_local->is_internal_fop = _gf_false; - /*Dont record counters*/ + /*Don't record counters*/ CTR_DB_REC(ctr_local).do_record_counters = _gf_false; /*Don't record time at all*/ CTR_DB_REC(ctr_local).do_record_times = _gf_false; @@ -135,7 +135,7 @@ ctr_lookup_wind(call_frame_t *frame, /* Since we are in lookup we can ignore errors while * Inserting in the DB, because there may be many * to write to the DB attempts for healing. - * We dont want to log all failed attempts and + * We don't want to log all failed attempts and * bloat the log*/ ctr_local->gfdb_db_record.ignore_errors = _gf_true; } @@ -216,7 +216,7 @@ ctr_lookup_cbk (call_frame_t *frame, void *cookie, xlator_t *this, CTR_IS_DISABLED_THEN_GOTO(this, out); - /* if the lookup failed lookup dont do anything*/ + /* if the lookup failed lookup don't do anything*/ if (op_ret == -1) { gf_msg_trace (this->name, 0, "lookup failed with %s", strerror (op_errno)); @@ -229,7 +229,7 @@ ctr_lookup_cbk (call_frame_t *frame, void *cookie, xlator_t *this, } /* if frame local was not set by the ctr_lookup() - * so dont so anything*/ + * so don't so anything*/ if (!frame->local) { goto out; } @@ -329,7 +329,7 @@ ctr_lookup (call_frame_t *frame, xlator_t *this, GF_ASSERT(frame); GF_ASSERT(frame->root); - /* Dont handle nameless lookups*/ + /* Don't handle nameless lookups*/ if (!loc->parent || !loc->name) goto out; @@ -941,7 +941,7 @@ ctr_rename (call_frame_t *frame, xlator_t *this, loc_t *oldloc, "Failed to insert rename wind"); } else { /* We are doing updation of hard link in inode context in wind - * As we dont get the "inode" in the call back for rename */ + * As we don't get the "inode" in the call back for rename */ ret = update_hard_link_ctx (frame, this, oldloc->inode); if (ret) { gf_msg (this->name, GF_LOG_ERROR, 0, @@ -950,13 +950,13 @@ ctr_rename (call_frame_t *frame, xlator_t *this, loc_t *oldloc, goto out; } - /* If the newloc has an inode. i.e aquiring hardlink of an + /* If the newloc has an inode. i.e acquiring hardlink of an * exisitng file i.e overwritting a file. * */ if (newloc->inode) { /* Getting the ctr inode context variable for - * inode whose hardlink will be aquired during + * inode whose hardlink will be acquired during * the rename * */ ctr_xlator_ctx = get_ctr_xlator_ctx (this, @@ -1106,7 +1106,7 @@ ctr_unlink (call_frame_t *frame, xlator_t *this, "Failed to insert unlink wind"); } else { /* We are doing delete of hard link in inode context in wind - * As we dont get the "inode" in the call back for rename */ + * As we don't get the "inode" in the call back for rename */ ret = delete_hard_link_ctx (frame, this, loc->inode); if (ret) { gf_msg (this->name, GF_LOG_ERROR, 0, diff --git a/xlators/features/changetimerecorder/src/ctr-helper.c b/xlators/features/changetimerecorder/src/ctr-helper.c index 822526ca833..5a409c2b65e 100644 --- a/xlators/features/changetimerecorder/src/ctr-helper.c +++ b/xlators/features/changetimerecorder/src/ctr-helper.c @@ -198,7 +198,7 @@ extract_sql_params(xlator_t *this, dict_t *params_dict) SET_DB_PARAM_TO_DICT(this->name, params_dict, GFDB_SQL_PARAM_DBPATH, db_full_path, ret, out); - /*Extact rest of the sql params*/ + /*Extract rest of the sql params*/ ret = gfdb_set_sql_params(this->name, this->options, params_dict); if (ret) { gf_msg (GFDB_DATA_STORE, GF_LOG_ERROR, 0, diff --git a/xlators/features/changetimerecorder/src/ctr-helper.h b/xlators/features/changetimerecorder/src/ctr-helper.h index 4fd4f745f4d..9232c16c5a9 100644 --- a/xlators/features/changetimerecorder/src/ctr-helper.h +++ b/xlators/features/changetimerecorder/src/ctr-helper.h @@ -76,7 +76,7 @@ typedef struct gf_ctr_private { * but currently we record only file inode information. * * is_internal_fop in gf_ctr_local will tell us if this is a internal fop and - * take special/no action. We dont record change/acces times or increement heat + * take special/no action. We don't record change/access times or increement heat * counter for internal fops from rebalancer. * */ typedef struct gf_ctr_local { @@ -146,7 +146,7 @@ free_ctr_local (gf_ctr_local_t *ctr_local) * ****************************************************************************/ /* - * Context Carrier structures are used to carry relavent information about + * Context Carrier structures are used to carry relevant information about * inodes and links from the fops calls to the ctr_insert_wind. * These structure just have pointers to the original data and donot * do a deep copy of any data. This info is deep copied to |