Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP

Loading…

Fix merge logging bug introduced by bs-merge-expiration-change branch #92

Merged
merged 1 commit into from

2 participants

@slfritchie
Owner

Reviewers: one of? @bsparrow435 @evanmcc @gburd

Fix merge logging bug (badmatch) introduced by bs-merge-expiration-change branch

@slfritchie slfritchie merged commit 2abb985 into from
@engelsanchez engelsanchez deleted the branch
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
This page is out of date. Refresh to see the latest.
Showing with 3 additions and 3 deletions.
  1. +3 −3 src/bitcask_merge_worker.erl
View
6 src/bitcask_merge_worker.erl
@@ -132,14 +132,14 @@ do_merge(Args) ->
Start = os:timestamp(),
Result = (catch apply(bitcask, merge, Args)),
ElapsedSecs = timer:now_diff(os:timestamp(), Start) / 1000000,
- [_,_,{Pargs,_}] = Args,
+ [_,_,Args3] = Args,
case Result of
ok ->
error_logger:info_msg("Merged ~p in ~p seconds.\n",
- [Pargs, ElapsedSecs]);
+ [Args3, ElapsedSecs]);
{Error, Reason} when Error == error; Error == 'EXIT' ->
error_logger:error_msg("Failed to merge ~p: ~p\n",
- [Pargs, Reason])
+ [Args3, Reason])
end;
false ->
ok
Something went wrong with that request. Please try again.