Alexey Serbin has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/21627 )

Change subject: [fs] check block size match only for block managers
......................................................................

[fs] check block size match only for block managers

Since the block size-related constraints affect only the implementation
of log block managers, this patch enables the extra sanity check in
DirInstanceMetadataFile::LoadFromDisk() only when Kudu's data is
controlled by 'log' or 'logr' block manager.

This allows for running Kudu test clusters with --block_manager=file
on Linux machines when keeping the data on a ZFS mount.

NOTE: file block manager shouldn't be used in production environments

Change-Id: I358214f4bc93fbf19bb245e3accc909943ae2f00
Reviewed-on: http://gerrit.cloudera.org:8080/21627
Reviewed-by: Yingchun Lai <laiyingc...@apache.org>
Tested-by: Alexey Serbin <ale...@apache.org>
---
M src/kudu/fs/dir_util.cc
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Yingchun Lai: Looks good to me, approved
  Alexey Serbin: Verified

--
To view, visit http://gerrit.cloudera.org:8080/21627
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I358214f4bc93fbf19bb245e3accc909943ae2f00
Gerrit-Change-Number: 21627
Gerrit-PatchSet: 2
Gerrit-Owner: Alexey Serbin <ale...@apache.org>
Gerrit-Reviewer: Abhishek Chennaka <achenn...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <ale...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Yingchun Lai <laiyingc...@apache.org>

Reply via email to