Hello Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/8767
to look at the new patch set (#2).
Change subject: tablet_copy_client-test: remove block verification check
......................................................................
tablet_copy_client-test: remove block verification check
The test copies a block from one server to another. Before doing so, the
copy client ensures that the block it's about to copy over does not
exist in the current FS, and doing so entails doing a block lookup.
The problem is that since 8a81f4ff18ee30951a25bab247fa5a3c20058e49, the
client's local FS has two data directories, while the remote tablet
server has three.
When running with --block_manager=file, block IDs are prefixed with the
index of the directory they are in. This is problematic because for
safety, block lookups DCHECK that the index of the block makes sense
with the number of data directories. Given the mentioned configurations,
when looking up a block in the third directory on the remote tserver,
the local FS would hit the DCHECK.
The solution in this patch is to make the number of directories in the
local FS equal the number of directories in the remove tserver.
Change-Id: I4c65a10a2a0f32ad0ecb467073dbec9965d2b093
---
M src/kudu/tserver/tablet_copy-test-base.h
M src/kudu/tserver/tablet_copy_client-test.cc
2 files changed, 11 insertions(+), 6 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/67/8767/2
--
To view, visit http://gerrit.cloudera.org:8080/8767
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I4c65a10a2a0f32ad0ecb467073dbec9965d2b093
Gerrit-Change-Number: 8767
Gerrit-PatchSet: 2
Gerrit-Owner: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins
