Repository: incubator-impala
Updated Branches:
  refs/heads/master 8c8a1a65e -> 9e03e9332


IMPALA-5863: Include-what-you-use for Kudu client

A recent commit in Kudu removed some unnecessary includes from Kudu
client headers to reduce compile times. We were implicitly relying on
those includes, so before we upgrade to a newer version of the client
we need to make the includes explicit on our side to avoid breaking
compilation.

Change-Id: Ibd041b783a9b7764f4b251291e0be5ed000485ce
Reviewed-on: http://gerrit.cloudera.org:8080/7872
Reviewed-by: Matthew Jacobs <[email protected]>
Tested-by: Impala Public Jenkins


Project: http://git-wip-us.apache.org/repos/asf/incubator-impala/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-impala/commit/a48960e2
Tree: http://git-wip-us.apache.org/repos/asf/incubator-impala/tree/a48960e2
Diff: http://git-wip-us.apache.org/repos/asf/incubator-impala/diff/a48960e2

Branch: refs/heads/master
Commit: a48960e2eb9f953d56d67a4fd126b70c2ffc697f
Parents: 8c8a1a6
Author: Thomas Tauber-Marshall <[email protected]>
Authored: Tue Aug 29 10:19:04 2017 -0700
Committer: Impala Public Jenkins <[email protected]>
Committed: Wed Aug 30 21:10:45 2017 +0000

----------------------------------------------------------------------
 be/src/exec/kudu-table-sink.cc     | 1 +
 be/src/exec/kudu-util.cc           | 1 +
 be/src/exprs/kudu-partition-expr.h | 1 +
 3 files changed, 3 insertions(+)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-impala/blob/a48960e2/be/src/exec/kudu-table-sink.cc
----------------------------------------------------------------------
diff --git a/be/src/exec/kudu-table-sink.cc b/be/src/exec/kudu-table-sink.cc
index 87eeb84..f01a373 100644
--- a/be/src/exec/kudu-table-sink.cc
+++ b/be/src/exec/kudu-table-sink.cc
@@ -17,6 +17,7 @@
 
 #include "exec/kudu-table-sink.h"
 
+#include <kudu/client/write_op.h>
 #include <sstream>
 #include <thrift/protocol/TDebugProtocol.h>
 

http://git-wip-us.apache.org/repos/asf/incubator-impala/blob/a48960e2/be/src/exec/kudu-util.cc
----------------------------------------------------------------------
diff --git a/be/src/exec/kudu-util.cc b/be/src/exec/kudu-util.cc
index e5a822a..320a77d 100644
--- a/be/src/exec/kudu-util.cc
+++ b/be/src/exec/kudu-util.cc
@@ -23,6 +23,7 @@
 
 #include <kudu/client/callbacks.h>
 #include <kudu/client/schema.h>
+#include <kudu/common/partial_row.h>
 
 #include "common/logging.h"
 #include "common/names.h"

http://git-wip-us.apache.org/repos/asf/incubator-impala/blob/a48960e2/be/src/exprs/kudu-partition-expr.h
----------------------------------------------------------------------
diff --git a/be/src/exprs/kudu-partition-expr.h 
b/be/src/exprs/kudu-partition-expr.h
index 015d6dd..90be8d0 100644
--- a/be/src/exprs/kudu-partition-expr.h
+++ b/be/src/exprs/kudu-partition-expr.h
@@ -19,6 +19,7 @@
 #define IMPALA_EXPRS_KUDU_PARTITION_EXPR_H_
 
 #include <kudu/client/client.h>
+#include <kudu/common/partial_row.h>
 
 #include "exprs/scalar-expr.h"
 

Reply via email to