Will Berkeley has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/11367 )

Change subject: [tablet] Improve logging of maintenance ops
......................................................................


Patch Set 3:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/11367/3/src/kudu/tablet/tablet.cc
File src/kudu/tablet/tablet.cc:

http://gerrit.cloudera.org:8080/#/c/11367/3/src/kudu/tablet/tablet.cc@1651
PS3, Line 1651:   LOG_WITH_PREFIX(INFO) << op_name << " successful on " << 
drsw.rows_written_count()
              :                         << " rows " << "(" << 
drsw.drs_written_count() << " rowsets, "
              :                         << drsw.written_size() << " bytes)";
              :   LOG_WITH_PREFIX(INFO) << Substitute("$0 successful on $1 rows 
($2 rowsets, $3 bytes)",
              :                                       op_name,
              :                                       drsw.rows_written_count(),
              :                                       drsw.drs_written_count(),
              :                                       drsw.written_size());
> Are these duplicates?  Seems like the first one should be removed.
Whoops.



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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I43883001c5a1c72ff1ca0c1bc84d24a8533e3891
Gerrit-Change-Number: 11367
Gerrit-PatchSet: 3
Gerrit-Owner: Will Berkeley <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Andrew Wong <[email protected]>
Gerrit-Reviewer: Andrew Wong <[email protected]>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Mike Percy <[email protected]>
Gerrit-Reviewer: Will Berkeley <[email protected]>
Gerrit-Comment-Date: Fri, 31 Aug 2018 18:07:10 +0000
Gerrit-HasComments: Yes

Reply via email to