From 5dbd7a2bc2b8e98ec115c38e237d08dfe3d384dd Mon Sep 17 00:00:00 2001 From: martinsumner Date: Thu, 16 Mar 2017 08:43:18 +0000 Subject: [PATCH] Check query out of range It doesn't work - so protecting against it in fetch_range is pointless, will blow up in lookup_slots --- src/leveled_sst.erl | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/src/leveled_sst.erl b/src/leveled_sst.erl index 05afac6..489715e 100644 --- a/src/leveled_sst.erl +++ b/src/leveled_sst.erl @@ -319,7 +319,7 @@ reader(get_maxsequencenumber, _From, State) -> Summary = State#state.summary, {reply, Summary#summary.max_sqn, reader, State}; reader(print_timings, _From, State) -> - io:format(user, "Timings of ~w~n", [State#state.sst_timings]), + io:format(user, "~nTimings of ~w~n", [State#state.sst_timings]), {reply, ok, reader, State#state{sst_timings = undefined}}; reader({set_for_delete, Penciller}, _From, State) -> leveled_log:log("SST06", [State#state.filename]), @@ -1711,6 +1711,11 @@ additional_range_test() -> R7 = sst_getkvrange(P1, SK, Slot1EK, 2), ?assertMatch(IK1, R7). + % Testing beyond end (should never happen if manifest behaves) + % Test blows up anyway + % R8 = sst_getkvrange(P1, element(1, PastEKV), element(1, PastEKV), 2), + % ?assertMatch([], R8). + simple_persisted_test() ->