Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP

Loading…

remove arbitrary 120-char limit on log_needs_merge messages #45

Merged
merged 1 commit into from

2 participants

@joecaswell
Collaborator

lager already chops log messages to 4kb

Sample entry for a single file exceeds 120 chars:
[{"./data/bitcask/bitcask/411047335499316445744786359201454599278231027712/1315.bitcask.data",[{oldest_tstamp,1342805897,1343043808}]}]

@jonmeredith
Owner

+1

@jonmeredith jonmeredith merged commit 10c0023 into from
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 1 addition and 1 deletion.
  1. +1 −1  src/bitcask.erl
View
2  src/bitcask.erl
@@ -587,7 +587,7 @@ needs_merge(Ref) ->
%% recv this information and decide if they want it
case get_opt(log_needs_merge, State#bc_state.opts) of
true ->
- error_logger:info_msg("~p needs_merge: ~120p\n",
+ error_logger:info_msg("~p needs_merge: ~p\n",
[State#bc_state.dirname, MergableFiles]);
_ ->
ok
Something went wrong with that request. Please try again.