Skip to content

Commit

Permalink
Push out new map before leaving cluster when rebalancing
Browse files Browse the repository at this point in the history
Previously if the node running the orchestrator was being removed, we
could end up being left with an old map.

Change-Id: Ic84d654c38f009d6e849426ef89cf0cb335483f6
Reviewed-on: http://review.northscale.com:8080/809
Reviewed-by: Steve Yen <steve.yen@gmail.com>
Tested-by: Steve Yen <steve.yen@gmail.com>
  • Loading branch information
Sean Lynch authored and steveyen committed Jun 22, 2010
1 parent 5631556 commit 5399dbd
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions src/ns_orchestrator.erl
Original file line number Diff line number Diff line change
Expand Up @@ -272,6 +272,8 @@ do_rebalance(Bucket, KeepNodes, EjectNodes, Map) ->
ns_bucket:set_servers(Bucket, KeepNodes),
ns_bucket:set_map(Bucket, Map5),
update_progress(Bucket, AllNodes, 0.9),
%% Push out the config with the new map in case this node is being removed
ns_config_rep:push(),
maybe_stop(),
ns_cluster_membership:deactivate(EjectNodes),
%% Leave myself last
Expand Down

0 comments on commit 5399dbd

Please sign in to comment.