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
......................................................................
Re-enable IWYU checks on HMS module
IWYU previously had some issues with the HMS module[1], probably due to
the Thrift generated classes. Now that IWYU has been upgraded[2] this is
an attempt to re-enable the IWYU checks. This commit also includes some
associated fixups.
[1]:
https://github.com/apache/kudu/commit/c09a6296181ecbb7d91687536f70fdbaac9fff5e
[2]:
https://github.com/apache/kudu/commit/bba339b11795d84da9c9d1473366ca561730c141
Change-Id: I63e5f1ca6a480f22ede83acde6baea8e2f563e8a
---
M build-support/iwyu/iwyu-filter.awk
M src/kudu/hms/hms_catalog-test.cc
M src/kudu/hms/hms_catalog.cc
M src/kudu/hms/hms_client-test.cc
M src/kudu/hms/hms_client.cc
M src/kudu/hms/mini_hms.cc
M src/kudu/hms/mini_hms.h
M src/kudu/integration-tests/master_hms-itest.cc
M src/kudu/master/catalog_manager.cc
M src/kudu/master/catalog_manager.h
10 files changed, 43 insertions(+), 46 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/02/10102/2
--
To view, visit http://gerrit.cloudera.org:8080/10102
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I63e5f1ca6a480f22ede83acde6baea8e2f563e8a
Gerrit-Change-Number: 10102
Gerrit-PatchSet: 2
Gerrit-Owner: Dan Burkert <danburk...@apache.org>
Gerrit-Reviewer: Adar Dembo <a...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com>
Gerrit-Reviewer: Dan Burkert <danburk...@apache.org>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <t...@apache.org>
