Dan Burkert has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/10075 )

Change subject: KUDU-2191: Metadata Upgrade Tool
......................................................................


Patch Set 3:

> Patch Set 3:
>
> > > Build Failed
>  > >
>  > > http://jenkins.kudu.apache.org/job/kudu-gerrit/13030/ : FAILURE
>  >
>  > Not sure why the jenkin's IWYU build gave errors and the output
>  > does not quite make sense. I built the same patch on a Linux
>  > machine and IWYU didn't complain anything.
>
> The reason might be some pollution in the Jenkins workspaces -- recently 
> there was a patch switching to a new LLVM and IWYU.  So, it might happen that 
> the IWYU build happened at some Jenkins slave with older LLVM/IWYU binary, 
> while at your build machine you saw output from the newest LLVM/IWYU.

Alexey, this has been a consistent issue with any patch touching HMS/thrift 
code.  See https://gerrit.cloudera.org/c/10102/ for an example.

Hao, add the new files to the iwyu-filter in order to make forward progress.


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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I1c8488e1c7b04b8bedc31c83b1496b53bae49cb3
Gerrit-Change-Number: 10075
Gerrit-PatchSet: 3
Gerrit-Owner: Hao Hao <hao....@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com>
Gerrit-Reviewer: Dan Burkert <danburk...@apache.org>
Gerrit-Reviewer: Hao Hao <hao....@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Tidy Bot
Gerrit-Comment-Date: Thu, 19 Apr 2018 19:51:01 +0000
Gerrit-HasComments: No

Reply via email to