[kudu-CR] Re-enable IWYU checks on HMS module

2018-04-24 Thread Alexey Serbin (Code Review)
Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/10102 ) Change subject: Re-enable IWYU checks on HMS module .. Re-enable IWYU checks on HMS module IWYU previously had some issues

[kudu-CR] Re-enable IWYU checks on HMS module

2018-04-24 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/10102 ) Change subject: Re-enable IWYU checks on HMS module .. Patch Set 3: Code-Review+2 -- To view, visit

[kudu-CR] Re-enable IWYU checks on HMS module

2018-04-24 Thread Alexey Serbin (Code Review)
Alexey Serbin has removed Kudu Jenkins from this change. ( http://gerrit.cloudera.org:8080/10102 ) Change subject: Re-enable IWYU checks on HMS module .. Removed reviewer Kudu Jenkins with the following votes: * Verified-1

[kudu-CR] Re-enable IWYU checks on HMS module

2018-04-24 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/10102 ) Change subject: Re-enable IWYU checks on HMS module .. Patch Set 3: > > Rebased. I'm a little confused, because I'm not getting iwyu

[kudu-CR] Re-enable IWYU checks on HMS module

2018-04-24 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/10102 ) Change subject: Re-enable IWYU checks on HMS module .. Patch Set 3: > Rebased. I'm a little confused, because I'm not getting iwyu

[kudu-CR] Re-enable IWYU checks on HMS module

2018-04-24 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/10102 ) Change subject: Re-enable IWYU checks on HMS module .. Patch Set 3: Rebased. I'm a little confused, because I'm not getting iwyu

[kudu-CR] Re-enable IWYU checks on HMS module

2018-04-24 Thread Dan Burkert (Code Review)
Hello Alexey Serbin, Kudu Jenkins, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10102 to look at the new patch set (#3). Change subject: Re-enable IWYU checks on HMS module

[kudu-CR] Re-enable IWYU checks on HMS module

2018-04-24 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/10102 ) Change subject: Re-enable IWYU checks on HMS module .. Patch Set 2: Is this still an issue after

[kudu-CR] Re-enable IWYU checks on HMS module

2018-04-18 Thread Dan Burkert (Code Review)
Hello Alexey Serbin, Kudu Jenkins, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10102 to look at the new patch set (#2). Change subject: Re-enable IWYU checks on HMS module

[kudu-CR] Re-enable IWYU checks on HMS module

2018-04-18 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/10102 ) Change subject: Re-enable IWYU checks on HMS module .. Patch Set 1: Code-Review+2 -- To view, visit

[kudu-CR] Re-enable IWYU checks on HMS module

2018-04-18 Thread Dan Burkert (Code Review)
Hello Adar Dembo, Todd Lipcon, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/10102 to review the following change. Change subject: Re-enable IWYU checks on HMS module .. Re-enable IWYU