Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/8605 )
Change subject: tablet: mark delta tracker read-only on error ...................................................................... Patch Set 7: (2 comments) http://gerrit.cloudera.org:8080/#/c/8605/6/src/kudu/tablet/delta_tracker.cc File src/kudu/tablet/delta_tracker.cc: http://gerrit.cloudera.org:8080/#/c/8605/6/src/kudu/tablet/delta_tracker.cc@355 PS6, Line 355: CheckWritableUnlocked( > nit: perhaps rename to CheckWritableUnlocked() Done http://gerrit.cloudera.org:8080/#/c/8605/6/src/kudu/tablet/delta_tracker.cc@740 PS6, Line 740: read_only_ = true; > CHECK(s.IsDiskFailure()) << LogPrefix() << s.ToString(); Done -- To view, visit http://gerrit.cloudera.org:8080/8605 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ib950048e9cd0929a10714ab1cc2bd829835afced Gerrit-Change-Number: 8605 Gerrit-PatchSet: 7 Gerrit-Owner: Andrew Wong <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Mike Percy <[email protected]> Gerrit-Reviewer: Todd Lipcon <[email protected]> Gerrit-Comment-Date: Tue, 21 Nov 2017 22:31:10 +0000 Gerrit-HasComments: Yes
