helifu has posted comments on this change. ( http://gerrit.cloudera.org:8080/13456 )
Change subject: [tablet] Support accurate count of rows ...................................................................... Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/13456/1/src/kudu/tablet/compaction-test.cc File src/kudu/tablet/compaction-test.cc: http://gerrit.cloudera.org:8080/#/c/13456/1/src/kudu/tablet/compaction-test.cc@a48 PS1, Line 48: It seems there is something wrong with iwyu. 2019-05-29 21:46:08,999 INFO: Checking 22 file(s)... >>> Fixing #includes in >>> '/mnt/ddb/2/helif/apache/kudu/src/kudu/tablet/compaction-test.cc' @@ -47,7 +47,6 @@ #include "kudu/consensus/opid.pb.h" #include "kudu/consensus/opid_util.h" #include "kudu/fs/block_id.h" -#include "kudu/fs/block_manager.h" #include "kudu/fs/fs_manager.h" #include "kudu/fs/log_block_manager.h" #include "kudu/gutil/casts.h" IWYU would have edited 1 files on your behalf. CMakeFiles/iwyu.dir/build.make:57: recipe for target 'CMakeFiles/iwyu' failed make[3]: *** [CMakeFiles/iwyu] Error 1 CMakeFiles/Makefile2:392: recipe for target 'CMakeFiles/iwyu.dir/all' failed make[2]: *** [CMakeFiles/iwyu.dir/all] Error 2 CMakeFiles/Makefile2:399: recipe for target 'CMakeFiles/iwyu.dir/rule' failed make[1]: *** [CMakeFiles/iwyu.dir/rule] Error 2 Makefile:305: recipe for target 'iwyu' failed make: *** [iwyu] Error 2 hzhelifu@db-87:/mnt/ddb/2/helif/apache/kudu/build/release$ lsb_release -a No LSB modules are available. Distributor ID: Debian Description: Debian GNU/Linux 8.9 (jessie) Release: 8.9 Codename: jessie -- To view, visit http://gerrit.cloudera.org:8080/13456 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I2e6378e289bb85024c29e96c2b153fc417ed6412 Gerrit-Change-Number: 13456 Gerrit-PatchSet: 1 Gerrit-Owner: helifu <hzhel...@corp.netease.com> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: helifu <hzhel...@corp.netease.com> Gerrit-Comment-Date: Wed, 29 May 2019 13:50:10 +0000 Gerrit-HasComments: Yes