You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kudu.apache.org by ha...@apache.org on 2018/08/22 00:59:00 UTC
[1/2] kudu git commit: [hms] use IPv4 for mini HMS
Repository: kudu
Updated Branches:
refs/heads/master 60020cf32 -> 9ad90eebe
[hms] use IPv4 for mini HMS
This patch explictly uses IPv4 protocol in the JVM environment options
that are passed to the mini HMS, to avoid IPv6 being accidentally used,
because IPv6 is not supported in Hadoop yet (https://wiki.apache.org/hadoop/HadoopIPv6).
Change-Id: I0ceeccb45f3cdd13ccdc5ed77e0675ae5442f808
Reviewed-on: http://gerrit.cloudera.org:8080/11276
Reviewed-by: Adar Dembo <ad...@cloudera.com>
Tested-by: Kudu Jenkins
Reviewed-by: Dan Burkert <da...@apache.org>
Project: http://git-wip-us.apache.org/repos/asf/kudu/repo
Commit: http://git-wip-us.apache.org/repos/asf/kudu/commit/43414452
Tree: http://git-wip-us.apache.org/repos/asf/kudu/tree/43414452
Diff: http://git-wip-us.apache.org/repos/asf/kudu/diff/43414452
Branch: refs/heads/master
Commit: 434144527bd68367f54f3a10e08b9500962dcd42
Parents: 60020cf
Author: Hao Hao <ha...@cloudera.com>
Authored: Mon Aug 20 12:21:31 2018 -0700
Committer: Hao Hao <ha...@cloudera.com>
Committed: Tue Aug 21 21:53:38 2018 +0000
----------------------------------------------------------------------
src/kudu/hms/mini_hms.cc | 2 ++
1 file changed, 2 insertions(+)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/kudu/blob/43414452/src/kudu/hms/mini_hms.cc
----------------------------------------------------------------------
diff --git a/src/kudu/hms/mini_hms.cc b/src/kudu/hms/mini_hms.cc
index 2ce2ea9..5472580 100644
--- a/src/kudu/hms/mini_hms.cc
+++ b/src/kudu/hms/mini_hms.cc
@@ -120,6 +120,8 @@ Status MiniHms::Start() {
// List of JVM environment options to pass to the HMS.
string java_options =
+ // Ensure IPv4 is used.
+ "-Djava.net.preferIPv4Stack=true "
// Make logging less verbose.
"-Dhive.log.level=WARN "
// Log to the console.
[2/2] kudu git commit: [hms] Add Hcatalog jars to the HMS classpath
Posted by ha...@apache.org.
[hms] Add Hcatalog jars to the HMS classpath
Due to the packaging differences of Hive distributions, the Hcatalog
jars may not be loaded into the classpath of mini HMS, which will cause
the ClassNotFoundException when using notification log event listener.
This patch explicitly adds these jars.
Change-Id: Ifed85fffdabb5af19f22e343843e012b009888a0
Reviewed-on: http://gerrit.cloudera.org:8080/11273
Reviewed-by: Adar Dembo <ad...@cloudera.com>
Tested-by: Kudu Jenkins
Reviewed-by: Dan Burkert <da...@apache.org>
Project: http://git-wip-us.apache.org/repos/asf/kudu/repo
Commit: http://git-wip-us.apache.org/repos/asf/kudu/commit/9ad90eeb
Tree: http://git-wip-us.apache.org/repos/asf/kudu/tree/9ad90eeb
Diff: http://git-wip-us.apache.org/repos/asf/kudu/diff/9ad90eeb
Branch: refs/heads/master
Commit: 9ad90eebe6324a30c17deaaf8593d82ba038db49
Parents: 4341445
Author: Hao Hao <ha...@cloudera.com>
Authored: Thu Aug 16 12:25:36 2018 -0700
Committer: Hao Hao <ha...@cloudera.com>
Committed: Tue Aug 21 21:53:44 2018 +0000
----------------------------------------------------------------------
src/kudu/hms/mini_hms.cc | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/kudu/blob/9ad90eeb/src/kudu/hms/mini_hms.cc
----------------------------------------------------------------------
diff --git a/src/kudu/hms/mini_hms.cc b/src/kudu/hms/mini_hms.cc
index 5472580..e52247d 100644
--- a/src/kudu/hms/mini_hms.cc
+++ b/src/kudu/hms/mini_hms.cc
@@ -116,7 +116,8 @@ Status MiniHms::Start() {
RETURN_NOT_OK(CreateCoreSite(tmp_dir));
// Comma-separated list of additional jars to add to the HMS classpath.
- string aux_jars = Substitute("$0/hms-plugin.jar", bin_dir);
+ string aux_jars = Substitute("$0/hms-plugin.jar,$1/hcatalog/share/hcatalog/*",
+ bin_dir, hive_home);
// List of JVM environment options to pass to the HMS.
string java_options =