diff options
author | Linus Nordberg <linus@nordu.net> | 2017-02-08 01:17:08 +0100 |
---|---|---|
committer | Linus Nordberg <linus@nordu.net> | 2017-02-08 01:17:08 +0100 |
commit | ce28e5b6ce3adf400b17f34e006426b411332af4 (patch) | |
tree | 38074c78d19336fae8fcdd8a7530013c577ff93c /merge | |
parent | ab86700d805fac3ed46e593266f1e8c75b01e63e (diff) |
Don't use 'if' after all.
'case' reads better IMO.
Diffstat (limited to 'merge')
-rw-r--r-- | merge/src/merge_sth.erl | 8 |
1 files changed, 3 insertions, 5 deletions
diff --git a/merge/src/merge_sth.erl b/merge/src/merge_sth.erl index 17a6a0d..a34179d 100644 --- a/merge/src/merge_sth.erl +++ b/merge/src/merge_sth.erl @@ -63,18 +63,16 @@ make_sth(CurSize, State) -> true = BackupQuorumSize =< length(MergeSecondaryNames), NewSize = lists:nth(BackupQuorumSize + 1, Sizes), lager:debug("new size at backup quorum ~B: ~B", [BackupQuorumSize, NewSize]), - Wait = - if - NewSize < CurSize -> + case NewSize < CurSize of + true -> lager:info("Waiting for enough backups to reach ~B, now at ~B.", [CurSize, NewSize]), 1; - true -> + false -> ok = do_make_sth(NewSize), max(1, round(application:get_env(plop, merge_delay, 600) / 60)) end, - Timer = erlang:start_timer(Wait * 1000, self(), make_sth), {noreply, State#state{timer = Timer}}. |