From 0dd60760c05f5291888a2fe4c8d2f860c97468bc Mon Sep 17 00:00:00 2001 From: Joseph Fernandes Date: Wed, 30 Dec 2015 16:53:25 +0530 Subject: [PATCH 03/86] tier/ctr/sql : Dafault values for sql cache and wal size Setting default values for sql cache and wal size cache : 12500 pages wal : 25000 pages 1 pages - 4096 bytes Porting this downstream 3.1.2 patch to 3.1.3 Label: DOWNSTREAM ONLY > Change-Id: Iae3927e021af2e3f7617d45f84e81de3b7d93f1c > BUG: 1282729 > Signed-off-by: Joseph Fernandes > Reviewed-on: https://code.engineering.redhat.com/gerrit/64642 > Reviewed-by: Dan Lambright > Tested-by: Dan Lambright Change-Id: Ib3cd951709dff25157371006637b8c0d881f5d61 Signed-off-by: Joseph Fernandes Reviewed-on: https://code.engineering.redhat.com/gerrit/70346 Reviewed-by: Nithya Balachandran Tested-by: Atin Mukherjee --- libglusterfs/src/gfdb/gfdb_sqlite3.h | 4 ++-- .../changetimerecorder/src/changetimerecorder.c | 4 ++-- xlators/mgmt/glusterd/src/glusterd-volume-set.c | 12 ++++++------ 3 files changed, 10 insertions(+), 10 deletions(-) diff --git a/libglusterfs/src/gfdb/gfdb_sqlite3.h b/libglusterfs/src/gfdb/gfdb_sqlite3.h index 9d0d996..e69251c 100644 --- a/libglusterfs/src/gfdb/gfdb_sqlite3.h +++ b/libglusterfs/src/gfdb/gfdb_sqlite3.h @@ -166,8 +166,8 @@ do {\ #define GF_SQL_DEFAULT_DBPATH "" #define GF_SQL_DEFAULT_PAGE_SIZE "4096" -#define GF_SQL_DEFAULT_CACHE_SIZE "1000" -#define GF_SQL_DEFAULT_WAL_AUTOCHECKPOINT "1000" +#define GF_SQL_DEFAULT_CACHE_SIZE "12500" +#define GF_SQL_DEFAULT_WAL_AUTOCHECKPOINT "25000" #define GF_SQL_DEFAULT_JOURNAL_MODE GF_SQL_JM_WAL #define GF_SQL_DEFAULT_SYNC GF_SQL_SYNC_OFF #define GF_SQL_DEFAULT_AUTO_VACUUM GF_SQL_AV_NONE diff --git a/xlators/features/changetimerecorder/src/changetimerecorder.c b/xlators/features/changetimerecorder/src/changetimerecorder.c index 5f3a074..3d2e78a 100644 --- a/xlators/features/changetimerecorder/src/changetimerecorder.c +++ b/xlators/features/changetimerecorder/src/changetimerecorder.c @@ -2299,11 +2299,11 @@ struct volume_options options[] = { }, { .key = {GFDB_SQL_PARAM_WAL_AUTOCHECK}, .type = GF_OPTION_TYPE_INT, - .default_value = "1000" + .default_value = "25000" }, { .key = {GFDB_SQL_PARAM_CACHE_SIZE}, .type = GF_OPTION_TYPE_INT, - .default_value = "1000" + .default_value = "12500" }, { .key = {GFDB_SQL_PARAM_PAGE_SIZE}, .type = GF_OPTION_TYPE_INT, diff --git a/xlators/mgmt/glusterd/src/glusterd-volume-set.c b/xlators/mgmt/glusterd/src/glusterd-volume-set.c index be624b7..1f59e36 100644 --- a/xlators/mgmt/glusterd/src/glusterd-volume-set.c +++ b/xlators/mgmt/glusterd/src/glusterd-volume-set.c @@ -2797,29 +2797,29 @@ struct volopt_map_entry glusterd_volopt_map[] = { }, { .key = "features.ctr-sql-db-cachesize", .voltype = "features/changetimerecorder", - .value = "1000", + .value = "12500", .option = "sql-db-cachesize", .validate_fn = validate_ctr_sql_params, .op_version = GD_OP_VERSION_3_7_7, .description = "Defines the cache size of the sqlite database of " "changetimerecorder xlator." "The input to this option is in pages." - "Each page is 4096 bytes. Default value is 1000 " - "pages i.e ~ 4 MB. " + "Each page is 4096 bytes. Default value is 12500 " + "pages i.e ~ 49 MB. " "The max value is 262144 pages i.e 1 GB and " "the min value is 1000 pages i.e ~ 4 MB. " }, { .key = "features.ctr-sql-db-wal-autocheckpoint", .voltype = "features/changetimerecorder", - .value = "1000", + .value = "25000", .option = "sql-db-wal-autocheckpoint", .validate_fn = validate_ctr_sql_params, .op_version = GD_OP_VERSION_3_7_7, .description = "Defines the autocheckpoint of the sqlite database of " " changetimerecorder. " "The input to this option is in pages. " - "Each page is 4096 bytes. Default value is 1000 " - "pages i.e ~ 4 MB." + "Each page is 4096 bytes. Default value is 25000 " + "pages i.e ~ 98 MB." "The max value is 262144 pages i.e 1 GB and " "the min value is 1000 pages i.e ~4 MB." }, -- 1.7.1