Todd Lipcon has submitted this change and it was merged. Change subject: cache: dynamically determine the number of cache shards ......................................................................
cache: dynamically determine the number of cache shards I noticed in a heavily cache-bound workload on a 48-core machine that there was a lot of contention on the cache shard spin locks. Manually bumping the number of cache shards to 64 instead of the default 16 seemed to reduce the amount of contention significantly. Change-Id: I2181ba4863065a554068a9c6d390b7f49e094abc Reviewed-on: http://gerrit.cloudera.org:8080/4292 Tested-by: Kudu Jenkins Reviewed-by: Adar Dembo <[email protected]> --- M src/kudu/util/cache.cc 1 file changed, 19 insertions(+), 7 deletions(-) Approvals: Adar Dembo: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/4292 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2181ba4863065a554068a9c6d390b7f49e094abc Gerrit-PatchSet: 2 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Todd Lipcon <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: David Ribeiro Alves <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon <[email protected]>
