Commit graph

202 commits

Author SHA1 Message Date
martinsumner
aa355a0aae Coverage cheat pt 2 2016-11-14 20:56:59 +00:00
martinsumner
8cbe2ef93a Coverage cheats
You juke the stats, and majors become colonels.  I've been here before
2016-11-14 20:43:38 +00:00
martinsumner
630f802780 Inker Close nastiness
Try to stop some of the potential deadlocking around Inker close and
prove that snapshots at higher Manifest SQNs can be ignored
2016-11-14 19:34:11 +00:00
martinsumner
75d6af75c6 Penciller review
The penciller attempt to close the L0 file if pending was unpredictable
in behaviour.  If a L0 file is still pending it will be lost - but this
is at least a predictable event.
2016-11-14 17:18:28 +00:00
martinsumner
eb76a6dbcd Multi-key test
Now failing with timeout coverage enabled.  100 keys is unnecessray so
reduce to 20 per loop, but suspect this is another issue
2016-11-14 12:43:45 +00:00
martinsumner
c0e1455430 Remove PromptDelete
Now that deleted journals are stored in waste for the retention period -
don't worry about deleting them
2016-11-14 11:40:02 +00:00
martinsumner
44738f7c75 Deferred Deletion of Journals
This allows for deleted journals to be retained for a period (the
waste_retnetion_period).  The idea being that a backup strategy can
ensure that all journals are backed up, even ones created and removed
from within a backup period - so that any restore pont is possible.

This is also a pre-cursor to removing some of the PromptDelete
complexity from the Inker Clerk - all compactions can prompt deletion as
deletion is now deferred.
2016-11-14 11:17:14 +00:00
martinsumner
dbb840d75e Ooops - test commenting 2016-11-09 22:06:02 +00:00
martinsumner
16c424cd7b Add tombstone to compaction problem
Minor test refinement
2016-11-09 20:03:44 +00:00
martinsumner
3d737dcf3c More sensible test assertion
Also better for test coverage stats
2016-11-09 19:38:02 +00:00
martinsumner
2458a90fb3 Remove overlapped case clause 2016-11-09 16:35:13 +00:00
martinsumner
feb3c2a524 Add recovr strategy test 2016-11-09 16:27:47 +00:00
martinsumner
3a77591da9 Removed unnecessary timeout condition from rolling state in CDB 2016-11-09 00:14:57 +00:00
martinsumner
0cf96e7198 Removed daft last function from SFT 2016-11-09 00:13:52 +00:00
martinsumner
0905ea51e2 Further corrupt file unit test for CDB 2016-11-08 23:07:03 +00:00
martinsumner
cc3cbc983b Tidy up closure of CDB Files 2016-11-08 22:43:22 +00:00
martinsumner
eedf5d85b0 Update readme text 2016-11-08 01:12:49 +00:00
martinsumner
8b8652e2d7 Coverage Tests
Add nonsense tests for nonsense coverage on standard methods.

Look at CDB search_hash_table - looks like it doubled-up on break-outs
so that one would never get hit
2016-11-08 01:03:09 +00:00
martinsumner
b43ec0de7e Fix the fix 2016-11-08 00:46:01 +00:00
martinsumner
6f0abea401 Stop unncessary handling of pending_delete in test 2016-11-08 00:44:51 +00:00
martinsumner
105d3b66f2 Add hash clash unit tests
Add unit tests for keys with clashing hashes
2016-11-07 23:53:14 +00:00
martinsumner
b47f8963d5 Speed of corrupted sft unit test
Test too slow when running with coverage
2016-11-07 19:20:46 +00:00
martinsumner
5a36f053d9 End-to-end testing of corrupted bloom
Add a unit test calling the API of a SFT file clerk with a corrupted
bloom
2016-11-07 18:26:55 +00:00
martinsumner
05338a1e85 Switch SFT to fsm
To amke it easier to context switch between the two file formats, make
the SFT file clerk a FSM like the CDB one.
2016-11-07 17:08:50 +00:00
martinsumner
aacf377f0a Provide project summary 2016-11-07 15:02:09 +00:00
martinsumner
b9090a3109 Testutil and pause 2016-11-07 12:19:17 +00:00
martinsumner
e2670e6e8c Testutil and Pause
Part of the testutil code is not recognising pause
2016-11-07 12:18:00 +00:00
martinsumner
8035583301 Comment review 2016-11-07 11:17:13 +00:00
martinsumner
079e713e16 Comment fiddling 2016-11-07 11:07:12 +00:00
martinsumner
6bf72849bc Comments reviewed 2016-11-07 11:02:38 +00:00
martinsumner
8f068c1280 Fixup API to allow for Tags to be passed
plus fixes to previous commits moving API functions into testutil
2016-11-07 10:42:49 +00:00
martinsumner
37c23a5b38 Shift pause out of leveled
Leveled will now signal the need for a pause due to back-pressure, but
not actually pause itself.  The hope is that in a riak implementation
this pause can be managed by the put_fsm, and so not lock the store.
2016-11-07 10:27:38 +00:00
martinsumner
4583460328 Clean API of Riak-specific Methods
Clena the API of Riak specific methods, and also resolve timing issue in
simple_server unit test.  Previously this would end up with missing data
(and a lower sequence number after start) because of the penciller_clerk
timeout being relatively large in the context of this test.  Now the
timeout has bene reduced the L0 slot is cleared by the time of the
close.  To make sure an extra sleep has been added as a precaution to
avoid any intermittent issues.
2016-11-07 10:11:57 +00:00
Martin Sumner
2d590c3077 More aggressive clerk 2016-11-05 17:50:28 +00:00
Martin Sumner
a7ed3e4b85 Trim dead branches
Also an experiment with altering the slowoffer_delay.
2016-11-05 15:59:31 +00:00
martinsumner
a73c233154 Correct the recording of excess work 2016-11-05 15:10:21 +00:00
martinsumner
376176eba3 Correct overlap in naming with Backlog 2016-11-05 14:35:01 +00:00
martinsumner
70dc637c97 Add slow offer support on write pressure
When there is write pressure on the penciller and it returns to the
bookie, the bookie will now punish the next PUT (and itself) with a
pause.  The longer the back-pressure state has been in place, the more
frequent the pauses
2016-11-05 14:31:10 +00:00
martinsumner
7f456fa993 Add back-pressure on work queue limit
Previously under heavy load, as long as L0 was being cleared, the ledger
woud keep accapting.  Now there is a formla limit on how far behind the
work queue (of compactions required at other levels) before the break is
applied on new updates coming in).
2016-11-05 14:04:45 +00:00
martinsumner
4556573a5c Rationalise logging on push_mem 2016-11-05 13:42:44 +00:00
martinsumner
9e410d65e3 Filename issue - not recording full filename in manifest
Broekn by change to get response on L0 completion, SFT was informing
penciller of the filename passed in (without extension), not the
completed one with the extension.
2016-11-05 12:49:15 +00:00
martinsumner
87b5bd0b18 Set Persisted SQN (regression)
As part of previous change had stopped setting the persisted SQN in the
ledger - which stopped journal compaction from working)
2016-11-05 12:03:21 +00:00
martinsumner
309ca2d6a1 Handle closing without an active handle (again) 2016-11-05 11:24:48 +00:00
martinsumner
61c6269200 Penciller back-pressure - Phase 1
There were issues with how the Penciller behaves under ehavy write
pressure - most particularly where there are a large number of keys per
update (i.e. 2i heavy objects).   Most immediately the attempt to chekc
whether the l0 file was ready slowed down the process of producing the
L0 file - so back-pressure created more back-pressure.

Going forward want to alter this most significantly as also the work
queue can build up unsustainably. there needs to be some pausing
prompted by the bookie on 'returned', and the use of 'returend when the
work queue exceeds a threshold.
2016-11-05 11:22:27 +00:00
martinsumner
34a90231e0 Prune dead branches 2016-11-04 19:33:11 +00:00
martinsumner
2299e1ce9d Prune unreachbale branches 2016-11-04 19:14:03 +00:00
martinsumner
a251f3eab0 Speed up query count test
Less individual querys to make count will speed up this taste, without
changing the nature of it
2016-11-04 18:20:00 +00:00
martinsumner
2b8a37439d Log refinement - logging process IDs 2016-11-04 17:28:04 +00:00
martinsumner
c306d49a85 CDB Logging
Setup CDB logging to use log base
2016-11-04 16:11:11 +00:00
martinsumner
479dc3ac80 Registering and releasing of Journal snapshots
Added a test of journal compaction with a registered snapshot and it
showed that the deleting of files did not correctly check the list of
registerd snapshots.  Corrected.
2016-11-04 15:56:57 +00:00