Skip to content

Commit

Permalink
Update for p4
Browse files Browse the repository at this point in the history
  • Loading branch information
martinsumner committed Sep 2, 2019
1 parent 04445cc commit 4d29808
Show file tree
Hide file tree
Showing 3 changed files with 20 additions and 4 deletions.
16 changes: 16 additions & 0 deletions RELEASE-NOTES.md
Original file line number Diff line number Diff line change
@@ -1,3 +1,19 @@
# Riak KV 2.9.0 Release Notes - Patch 4

There are a number of fixes in this patch:

Mitigation, and further monitoring, to assist in [an issue](https://github.com/basho/riak_kv/issues/1707) where a riak_kv_vnode may attempt to PUT an object with empty contents. This will now check for this case regardless of the allow_mult setting, and prompt a PUT failure without crashing the vnode.

Tictac AAE will now, as with standard AAE, exchange between primary vnodes only for a partition (and not backfill fallbacks). This change can be reversed by configuration, and this also helps with an issue with [delayed handoff due to Tictac AAE](https://github.com/basho/riak_kv/issues/1706).

A change to allow the management via riak attach of the tokens used to ensure that the AAE database is not flooded due to relative under-performance of this store when compared to the vnode store. This allows for admin activities (like coordinated rebuilds) to be managed free of lock issues related to token depletion. This is partial mitigation for a broader [issue](https://github.com/basho/riak/issues/981), and the change also includes improved monitoring of the time it takes to clear trees during the rebuild process to help further investigate the underlying cause of these issues.

A [refactoring of the code](https://github.com/basho/riak_kv/pull/1710) used when managing a GET request with HEAD and GET responses interspersed. This is a refactor to improve readability, dialyzer support and also enhance the predictability of behaviour under certain race conditions. There is no known issue resolved by this change.

An uplift to the leveled version to add in an extra log (on cache effectiveness) and resolve an intermittent test failure.

It is recommended that any 2.9.0 installations be upgraded to include this path, although if Tictac AAE is not used there is no immediate urgency to making the change.

# Riak KV 2.9.0 Release Notes - Patch 3

An [issue](https://github.com/martinsumner/leveled/issues/287) was discovered in leveled, whereby following a restart of Riak and a workload of fetch requests, the backend demanded excess amounts of binary heap references. Underlying was an issue with the use of sub-binary references during the lazy load of slot header information after a SST file process restart. This has been resolved, and with [greater control added](https://github.com/martinsumner/leveled/blob/0.9.18/priv/leveled.schema#L86-L93) to force the ledger contents into the page cache at startup.
Expand Down
2 changes: 1 addition & 1 deletion rebar.config
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@
{node_package, ".*", {git, "https://github.com/basho/node_package.git", {tag, "4.0.2"}}},
{lager_syslog, ".*", {git, "git://github.com/basho/lager_syslog.git", {tag, "3.0.3"}}},
{cluster_info, ".*", {git, "git://github.com/basho/cluster_info.git", {tag, "2.0.5"}}},
{riak_kv, ".*", {git, "git://github.com/basho/riak_kv.git", {tag, "riak_kv-2.9.0p4"}}},
{riak_kv, ".*", {git, "git://github.com/basho/riak_kv.git", {tag, "riak_kv-2.9.0p5"}}},
{riak_control, ".*", {git, "git://github.com/basho/riak_control.git", {branch, "develop-2.9"}}},
{riaknostic, ".*", {git, "git://github.com/basho/riaknostic.git", {tag, "2.1.7"}}},
{riak_auth_mods, ".*", {git, "git://github.com/basho/riak_auth_mods.git", {tag, "2.1.0"}}},
Expand Down
6 changes: 3 additions & 3 deletions rebar.config.lock
Original file line number Diff line number Diff line change
Expand Up @@ -108,10 +108,10 @@
"6bc5efe93bab327788bcde24374f16044b549dd8"}},
{leveled,".*",
{git,"https://github.com/martinsumner/leveled.git",
"e3913a6c074473d14ad2df3fce6ad125b87b5ac8"}},
"432fe71bf09987f52fc407a776c23e8eca802451"}},
{kv_index_tictactree,".*",
{git,"https://github.com/martinsumner/kv_index_tictactree.git",
"5869cff7c8345f3fc580d2db30a98a790769b736"}},
"b2423eaf11493eb51fc8089a843f8c24d951404b"}},
{protobuffs,".*",
{git,"https://github.com/basho/erlang_protobuffs.git",
"43c549e6ccd4524245190a906bb4344039bd2995"}},
Expand Down Expand Up @@ -141,7 +141,7 @@
"fdb62672000be1477e2c8b6d02289abe411baf91"}},
{riak_kv,".*",
{git,"git://github.com/basho/riak_kv.git",
"f829e65385adbd9e50f556fac698ee2ce2139430"}},
"841e8792ca2c1377af0c36932a9836f74d919562"}},
{erlydtl,".*",
{git,"git://github.com/erlydtl/erlydtl.git",
"d20b53f04837a1053ed18987f645cb60eae82453"}},
Expand Down

0 comments on commit 4d29808

Please sign in to comment.