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 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 Reviewed-on: http://gerrit.cloudera.org:8080/10102 Tested-by: Alexey Serbin <aser...@cloudera.com> Reviewed-by: Alexey Serbin <aser...@cloudera.com> --- 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.h 8 files changed, 43 insertions(+), 37 deletions(-) Approvals: Alexey Serbin: Looks good to me, approved; Verified -- 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: merged Gerrit-Change-Id: I63e5f1ca6a480f22ede83acde6baea8e2f563e8a Gerrit-Change-Number: 10102 Gerrit-PatchSet: 4 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: Todd Lipcon <t...@apache.org>