Merge pull request #5 from basho-labs/gsb-cache-size-est
Estimate a reasonable cache size if one isn't provided in app/env config for wterl.
This commit is contained in:
commit
85fadeec70
2 changed files with 44 additions and 4 deletions
|
@ -24,6 +24,8 @@
|
||||||
-behavior(temp_riak_kv_backend).
|
-behavior(temp_riak_kv_backend).
|
||||||
-author('Steve Vinoski <steve@basho.com>').
|
-author('Steve Vinoski <steve@basho.com>').
|
||||||
|
|
||||||
|
-compile([{parse_transform, lager_transform}]).
|
||||||
|
|
||||||
%% KV Backend API
|
%% KV Backend API
|
||||||
-export([api_version/0,
|
-export([api_version/0,
|
||||||
capabilities/1,
|
capabilities/1,
|
||||||
|
@ -111,7 +113,7 @@ start(Partition, Config) ->
|
||||||
{logging, true},
|
{logging, true},
|
||||||
{transactional, true},
|
{transactional, true},
|
||||||
{session_max, SessionMax},
|
{session_max, SessionMax},
|
||||||
{cache_size, "2GB"},
|
{cache_size, size_cache(Config)},
|
||||||
{sync, false}
|
{sync, false}
|
||||||
%% {verbose,
|
%% {verbose,
|
||||||
%% ["block", "shared_cache", "ckpt", "evict",
|
%% ["block", "shared_cache", "ckpt", "evict",
|
||||||
|
@ -135,8 +137,7 @@ start(Partition, Config) ->
|
||||||
session=SRef,
|
session=SRef,
|
||||||
partition=Partition}};
|
partition=Partition}};
|
||||||
{error, ConnReason}=ConnError ->
|
{error, ConnReason}=ConnError ->
|
||||||
lager:error("Failed to start wterl backend: ~p\n",
|
lager:error("Failed to start wterl backend: ~p\n", [ConnReason]),
|
||||||
[ConnReason]),
|
|
||||||
ConnError
|
ConnError
|
||||||
end;
|
end;
|
||||||
Error ->
|
Error ->
|
||||||
|
@ -476,6 +477,43 @@ fetch_status(Cursor, {ok, Stat}, Acc) ->
|
||||||
[What,Val|_] = [binary_to_list(B) || B <- binary:split(Stat, [<<0>>], [global])],
|
[What,Val|_] = [binary_to_list(B) || B <- binary:split(Stat, [<<0>>], [global])],
|
||||||
fetch_status(Cursor, wterl:cursor_next_value(Cursor), [{What,Val}|Acc]).
|
fetch_status(Cursor, wterl:cursor_next_value(Cursor), [{What,Val}|Acc]).
|
||||||
|
|
||||||
|
size_cache(Config) ->
|
||||||
|
Size =
|
||||||
|
case app_helper:get_prop_or_env(cache_size, Config, wterl) of
|
||||||
|
{ok, Value} ->
|
||||||
|
Value;
|
||||||
|
undefined ->
|
||||||
|
RunningApps = application:which_applications(),
|
||||||
|
FinalGuess =
|
||||||
|
case proplists:is_defined(sasl, RunningApps) andalso
|
||||||
|
proplists:is_defined(os_mon, RunningApps) of
|
||||||
|
true ->
|
||||||
|
Memory = memsup:get_system_memory_data(),
|
||||||
|
TotalRAM = proplists:get_value(system_total_memory, Memory),
|
||||||
|
FreeRAM = proplists:get_value(free_memory, Memory),
|
||||||
|
UsedByBeam = proplists:get_value(total, erlang:memory()),
|
||||||
|
Target = ((TotalRAM - UsedByBeam) div 3),
|
||||||
|
FirstGuess = (Target - (Target rem (1024 * 1024))),
|
||||||
|
SecondGuess =
|
||||||
|
case FirstGuess > FreeRAM of
|
||||||
|
true -> FreeRAM - (FreeRAM rem (1024 * 1024));
|
||||||
|
_ -> FirstGuess
|
||||||
|
end,
|
||||||
|
case SecondGuess < 1073741824 of %% < 1GB?
|
||||||
|
true -> "1GB";
|
||||||
|
false ->
|
||||||
|
ThirdGuess = SecondGuess div (1024 * 1024),
|
||||||
|
integer_to_list(ThirdGuess) ++ "MB"
|
||||||
|
end;
|
||||||
|
false ->
|
||||||
|
"1GB"
|
||||||
|
end,
|
||||||
|
application:set_env(wt, cache_size, FinalGuess),
|
||||||
|
lager:warning("Using best-guess cache size of ~p for WiredTiger storage backend.", [FinalGuess]),
|
||||||
|
FinalGuess
|
||||||
|
end,
|
||||||
|
Size.
|
||||||
|
|
||||||
%% ===================================================================
|
%% ===================================================================
|
||||||
%% EUnit tests
|
%% EUnit tests
|
||||||
%% ===================================================================
|
%% ===================================================================
|
||||||
|
|
|
@ -272,7 +272,9 @@ empty_check({Backend, State}) ->
|
||||||
}.
|
}.
|
||||||
|
|
||||||
setup({BackendMod, Config}) ->
|
setup({BackendMod, Config}) ->
|
||||||
%% Start the backend
|
lager:start(),
|
||||||
|
application:start(sasl),
|
||||||
|
application:start(os_mon),
|
||||||
{ok, S} = BackendMod:start(42, Config),
|
{ok, S} = BackendMod:start(42, Config),
|
||||||
{BackendMod, S}.
|
{BackendMod, S}.
|
||||||
|
|
||||||
|
|
Loading…
Reference in a new issue