Mahesh Reddy has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/22132


Change subject: KUDU-613: Don't modify ref count when moving entries
......................................................................

KUDU-613: Don't modify ref count when moving entries

The current implementation of the SLRU cache temporarily
changes the reference count of entries when moving between
segments of the cache (either during an upgrade or downgrade).
This can cause a concurrent Release call to an entry to think
it's the last reference to the entry and free it when really
it's just moving segments.

This patch changes this behavior by not modifying the ref
count of entries when moving between segments. A repro
scenario is added to cache-bench.

Change-Id: I643907612d43eba2c5f8dbc19d2f74f88bbca869
---
M src/kudu/util/cache-bench.cc
M src/kudu/util/slru_cache.cc
M src/kudu/util/slru_cache.h
3 files changed, 56 insertions(+), 31 deletions(-)



  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/32/22132/1
--
To view, visit http://gerrit.cloudera.org:8080/22132
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I643907612d43eba2c5f8dbc19d2f74f88bbca869
Gerrit-Change-Number: 22132
Gerrit-PatchSet: 1
Gerrit-Owner: Mahesh Reddy <[email protected]>

Reply via email to