Hello Dan Burkert, Todd Lipcon,

I'd like you to do a code review.  Please visit

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

to review the following change.

Change subject: log block manage: tighten up read only usage
......................................................................

log block manage: tighten up read only usage

Over time we had introduced a couple new file deletion/truncation sites that
weren't checking read_only_ before making destructive changes.

Found via code inspection.

Change-Id: I029527b78148d68a002a3c42af5a52f179089a95
---
M src/kudu/fs/log_block_manager.cc
1 file changed, 11 insertions(+), 7 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/83/5383/1
-- 
To view, visit http://gerrit.cloudera.org:8080/5383
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I029527b78148d68a002a3c42af5a52f179089a95
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Adar Dembo <[email protected]>
Gerrit-Reviewer: Dan Burkert <[email protected]>
Gerrit-Reviewer: Todd Lipcon <[email protected]>

Reply via email to