Kim Jin Chul has posted comments on this change. ( http://gerrit.cloudera.org:8080/8829 )
Change subject: IMPALA-5754: Rollback the exclusion of clang-tidy check for pcg-cpp ...................................................................... Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/8829/2/be/src/thirdparty/pcg-cpp-0.98/include/pcg_extras.hpp File be/src/thirdparty/pcg-cpp-0.98/include/pcg_extras.hpp: http://gerrit.cloudera.org:8080/#/c/8829/2/be/src/thirdparty/pcg-cpp-0.98/include/pcg_extras.hpp@599 PS2, Line 599: * warning: expansion of date or time macro is not reproducible The struct is not used in our code and pcg-cpp's sample and test codes. I thought comment out is better than removal of the code because someone needs this later. -- To view, visit http://gerrit.cloudera.org:8080/8829 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I591d30373cb13f0eb89afbe16d81b1d3fb783365 Gerrit-Change-Number: 8829 Gerrit-PatchSet: 2 Gerrit-Owner: Kim Jin Chul <jinc...@gmail.com> Gerrit-Reviewer: Jim Apple <jbapple-imp...@apache.org> Gerrit-Reviewer: Kim Jin Chul <jinc...@gmail.com> Gerrit-Comment-Date: Wed, 13 Dec 2017 13:38:35 +0000 Gerrit-HasComments: Yes