Permalink
Browse files

MB-8045: bumped default number of moves before compaction to 64

I.e. our perf data suggests that 64 doesn't blow up disk space usage
too much and saves a ton of time on rebalance.

Change-Id: Iac1f8a2caa3e97f6eaf1b2d427fb257f43e31e09
Reviewed-on: http://review.couchbase.org/25571
Reviewed-by: Aliaksey Kandratsenka <alkondratenko@gmail.com>
Tested-by: Aliaksey Kandratsenka <alkondratenko@gmail.com>
  • Loading branch information...
1 parent ba7f207 commit de9286980424d5aecebc402b98e1634e8b1e33aa Aliaksey Kandratsenka committed with alk Apr 9, 2013
Showing with 5 additions and 2 deletions.
  1. +3 −0 CHANGES
  2. +1 −1 src/menelaus_web.erl
  3. +1 −1 src/ns_vbucket_mover.erl
View
@@ -5,6 +5,9 @@ This file documents user-visible changes in Couchbase clustering & UI.
Between versions 2.0.1 and 2.0.2
-----------------------------------------
+* (MB-8045) Default value of rebalanceMovesBeforeCompaction was raised
+ to 64 for severe gains in rebalance time.
+
* (MB-7398) There's now full support of assigning symbolic hostnames
to cluster nodes that replaces old "manual" and kludgy
procedure. Now as part of node setup wizard there's input field to
View
@@ -2468,7 +2468,7 @@ build_internal_settings_kvs() ->
{index_pausing_disabled, rebalanceIndexPausingDisabled, false},
{rebalance_ignore_view_compactions, rebalanceIgnoreViewCompactions, false},
{rebalance_moves_per_node, rebalanceMovesPerNode, 1},
- {rebalance_moves_before_compaction, rebalanceMovesBeforeCompaction, 16},
+ {rebalance_moves_before_compaction, rebalanceMovesBeforeCompaction, 64},
{{couchdb, max_parallel_indexers}, maxParallelIndexers, <<>>},
{{couchdb, max_parallel_replica_indexers}, maxParallelReplicaIndexers, <<>>},
{max_bucket_count, maxBucketCount, 10},
View
@@ -22,7 +22,7 @@
-include_lib("eunit/include/eunit.hrl").
-define(MAX_MOVES_PER_NODE, ns_config_ets_dup:unreliable_read_key(rebalance_moves_per_node, 1)).
--define(MOVES_BEFORE_COMPACTION, ns_config_ets_dup:unreliable_read_key(rebalance_moves_before_compaction, 16)).
+-define(MOVES_BEFORE_COMPACTION, ns_config_ets_dup:unreliable_read_key(rebalance_moves_before_compaction, 64)).
-define(TAP_STATS_LOGGING_INTERVAL, 10*60*1000).

0 comments on commit de92869

Please sign in to comment.