Impala Public Jenkins has submitted this change and it was merged. Change subject: Remove spurious forward declarations and includes in coordinator.cc/h ......................................................................
Remove spurious forward declarations and includes in coordinator.cc/h While working on another patch, I noticed that a lot of includes and forward declarations were spurious and possibly the result of bit rot. This patch removes them and hopefully improves compile time a little. Testing: Made sure that Impala and the BE tests compile successfully. Change-Id: Id0afed224fad6a00698701487b51506d414f83ac Reviewed-on: http://gerrit.cloudera.org:8080/7482 Reviewed-by: Sailesh Mukil <[email protected]> Tested-by: Impala Public Jenkins --- M be/src/runtime/coordinator.cc M be/src/runtime/coordinator.h 2 files changed, 2 insertions(+), 22 deletions(-) Approvals: Impala Public Jenkins: Verified Sailesh Mukil: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/7482 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id0afed224fad6a00698701487b51506d414f83ac Gerrit-PatchSet: 3 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: Sailesh Mukil <[email protected]> Gerrit-Reviewer: Impala Public Jenkins Gerrit-Reviewer: Matthew Jacobs <[email protected]> Gerrit-Reviewer: Sailesh Mukil <[email protected]>
