David Ribeiro Alves has submitted this change and it was merged.

Change subject: Improve the debuggability of 
LogBlockContainer::CheckBlockRecord()
......................................................................


Improve the debuggability of LogBlockContainer::CheckBlockRecord()

We're getting an error come up in the flaky tests that is hard to debug
because we don't print the data file size, which is the condition tripping
the check. This also includes the filename of the data file where the
offending record was found in the printout.

Change-Id: I3e74537ea258e8c6a0ffa00b9f5d529a04ecd583
Reviewed-on: http://gerrit.cloudera.org:8080/4451
Reviewed-by: Todd Lipcon <t...@apache.org>
Tested-by: Kudu Jenkins
---
M src/kudu/fs/log_block_manager.cc
1 file changed, 3 insertions(+), 1 deletion(-)

Approvals:
  Todd Lipcon: Looks good to me, approved
  Kudu Jenkins: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I3e74537ea258e8c6a0ffa00b9f5d529a04ecd583
Gerrit-PatchSet: 3
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: David Ribeiro Alves <dral...@apache.org>
Gerrit-Reviewer: Adar Dembo <a...@cloudera.com>
Gerrit-Reviewer: David Ribeiro Alves <dral...@apache.org>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <t...@apache.org>

Reply via email to