Mike Percy has posted comments on this change. ( http://gerrit.cloudera.org:8080/10505 )
Change subject: KUDU-702 Add block IDs to more log messages ...................................................................... Patch Set 4: Code-Review+2 (1 comment) http://gerrit.cloudera.org:8080/#/c/10505/1/src/kudu/cfile/cfile_reader.cc File src/kudu/cfile/cfile_reader.cc: http://gerrit.cloudera.org:8080/#/c/10505/1/src/kudu/cfile/cfile_reader.cc@491 PS1, Line 491: LOG(WARNING) << "Unable to validate compressed block " << block_id().ToString() : << " at " << ptr.offset() << " of size " << block.size() << ": " : << s.ToString(); : return s; > Nit: the ordering here seems off to me, I think we should first identify th Done -- To view, visit http://gerrit.cloudera.org:8080/10505 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I895da1cc04ecbf006f412f06b31461b03072d32d Gerrit-Change-Number: 10505 Gerrit-PatchSet: 4 Gerrit-Owner: Anupama Gupta <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Anupama Gupta <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Mike Percy <[email protected]> Gerrit-Comment-Date: Tue, 29 May 2018 07:16:12 +0000 Gerrit-HasComments: Yes
