Hello Alexey Serbin, Kudu Jenkins, Adar Dembo,

I'd like you to reexamine a change. Please visit

    http://gerrit.cloudera.org:8080/14212

to look at the new patch set (#2).

Change subject: KUDU-2920: Block cache capacity validator shouldn't run on an 
NVM block cache
......................................................................

KUDU-2920: Block cache capacity validator shouldn't run on an NVM block
cache

Validator should fail for options:
--block_cache_type=DRAM
--block_cache_capacity_mb=1000000
and succeed for options:
--block_cache_type=NVM
--block_cache_capacity_mb=1000000
--nvm_cache_path=/path/to/your/tmp/dir
where 1000000 means "size of cache bigger than RAM" (1000GB)

Change-Id: I9f50e9dd901280b5c32576e43165292299922995
---
M src/kudu/cfile/block_cache.cc
M src/kudu/cfile/block_cache.h
M src/kudu/cfile/cfile-test.cc
3 files changed, 41 insertions(+), 17 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/12/14212/2
--
To view, visit http://gerrit.cloudera.org:8080/14212
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I9f50e9dd901280b5c32576e43165292299922995
Gerrit-Change-Number: 14212
Gerrit-PatchSet: 2
Gerrit-Owner: Volodymyr Verovkin <verjov...@cloudera.com>
Gerrit-Reviewer: Adar Dembo <a...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins (120)

Reply via email to