Elukey has submitted this change and it was merged. (
https://gerrit.wikimedia.org/r/368820 )
Change subject: hive: remove etc default config in favor of hive-env.sh
......................................................................
hive: remove etc default config in favor of hive-env.sh
Bug: T172107
Change-Id: I5c0c044ee982843443c6e2c5a2f1fa6ddc949a6f
---
M manifests/hive.pp
M manifests/hive/defaults.pp
M manifests/hive/metastore.pp
M manifests/hive/server.pp
A templates/hive/hive-env.sh.erb
D templates/hive/hive-metastore.default.erb
D templates/hive/hive-server2.default.erb
7 files changed, 34 insertions(+), 79 deletions(-)
Approvals:
Ottomata: Looks good to me, but someone else must approve
Elukey: Looks good to me, approved
jenkins-bot: Verified
diff --git a/manifests/hive.pp b/manifests/hive.pp
index 1f187e9..9f2c779 100644
--- a/manifests/hive.pp
+++ b/manifests/hive.pp
@@ -83,7 +83,14 @@
$hive_site_template = $cdh::hive::defaults::hive_site_template,
$hive_log4j_template = $cdh::hive::defaults::hive_log4j_template,
- $hive_exec_log4j_template =
$cdh::hive::defaults::hive_exec_log4j_template
+ $hive_exec_log4j_template =
$cdh::hive::defaults::hive_exec_log4j_template,
+ $hive_env_template = $cdh::hive::defaults::hive_env_template,
+
+ $hive_metastore_heap_size =
$cdh::hive::defaults::hive_metastore_heap_size,
+ $hive_server_heap_size = $cdh::hive::defaults::hive_server_heap_size,
+ $hive_metastore_jmx_port =
$cdh::hive::defaults::hive_metastore_jmx_port,
+ $hive_server_jmx_port = $cdh::hive::defaults::hive_server_jmx_port,
+
) inherits cdh::hive::defaults
{
Class['cdh::hadoop'] -> Class['cdh::hive']
@@ -117,6 +124,13 @@
$::fqdn => '0440',
default => '0444',
}
+ file { "${config_directory}/hive-env.sh":
+ content => template($hive_env_template),
+ mode => '0444',
+ owner => 'root',
+ group => 'hdfs',
+ require => Package['hive'],
+ }
file { "${config_directory}/hive-site.xml":
content => template($hive_site_template),
mode => $hive_site_mode,
diff --git a/manifests/hive/defaults.pp b/manifests/hive/defaults.pp
index 276077b..1513b2b 100644
--- a/manifests/hive/defaults.pp
+++ b/manifests/hive/defaults.pp
@@ -47,8 +47,11 @@
$hive_site_template = 'cdh/hive/hive-site.xml.erb'
$hive_log4j_template = 'cdh/hive/hive-log4j.properties.erb'
$hive_exec_log4j_template = 'cdh/hive/hive-exec-log4j.properties.erb'
+ $hive_env_template = 'cdh/hive/hive-env.sh.erb'
# Jmx ports
$hive_server_jmx_port = 9978
$hive_metastore_jmx_port = 9979
+ $hive_metastore_opts = '-Xmx2048m'
+ $hive_server_opts = '-Xmx2048m'
}
\ No newline at end of file
diff --git a/manifests/hive/metastore.pp b/manifests/hive/metastore.pp
index b519955..e8bfbf6 100644
--- a/manifests/hive/metastore.pp
+++ b/manifests/hive/metastore.pp
@@ -4,24 +4,15 @@
#
# == Parameters
# $port - Port on which hive-metastore listens. Default: undef
-# $heapsize - -Xmx in MB. Default: undef
#
class cdh::hive::metastore(
$port = undef,
- $heapsize = undef,
- $jmx_port = $cdh::hive::defaults::hive_metastore_jmx_port,
- $default_template = 'cdh/hive/hive-metastore.default.erb',
)
{
Class['cdh::hive'] -> Class['cdh::hive::metastore']
package { 'hive-metastore':
ensure => 'installed',
- }
-
- file { '/etc/default/hive-metastore':
- content => template($default_template),
- require => Package['hive-metastore'],
}
# If the metastore will use MySQL for storage, then
diff --git a/manifests/hive/server.pp b/manifests/hive/server.pp
index f3a6269..b95bdcf 100644
--- a/manifests/hive/server.pp
+++ b/manifests/hive/server.pp
@@ -6,13 +6,9 @@
#
# == Parameters
# $port - Port on which hive-server2 listens. Default: undef
-# $heapsize - -Xmx in MB. Default: undef
#
class cdh::hive::server(
$port = undef,
- $heapsize = undef,
- $jmx_port = $cdh::hive::defaults::hive_server_jmx_port,
- $default_template = 'cdh/hive/hive-server2.default.erb'
)
{
# cdh::hive::server requires hadoop client and configs are installed.
@@ -22,11 +18,6 @@
package { 'hive-server2':
ensure => 'installed',
alias => 'hive-server',
- }
-
- file { '/etc/default/hive-server2':
- content => template($default_template),
- require => Package['hive-server2'],
}
# sudo -u hdfs hdfs dfs -mkdir /user/hive
diff --git a/templates/hive/hive-env.sh.erb b/templates/hive/hive-env.sh.erb
new file mode 100644
index 0000000..c14f7a0
--- /dev/null
+++ b/templates/hive/hive-env.sh.erb
@@ -0,0 +1,16 @@
+#! /bin/bash
+# Note: This file is managed by Puppet
+
+if [ "$SERVICE" = "metastore" ]; then
+ export HADOOP_OPTS="$HADOOP_OPTS ${hive_metastore_opts}"
+ <% if @hive_metastore_jmx_port -%>
+ export HADOOP_OPTS="$HADOOP_OPTS
-Dcom.sun.management.jmxremote.port=<%= @hive_metastore_jmx_port %>
-Dcom.sun.management.jmxremote.ssl=false
-Dcom.sun.management.jmxremote.authenticate=false"
+ <% end -%>
+fi
+
+if [ "$SERVICE" = "hiveserver2" ]; then
+ export HADOOP_OPTS="$HADOOP_OPTS ${hive_server_opts}"
+ <% if @hive_server_jmx_port -%>
+ export HADOOP_OPTS="$HADOOP_OPTS
-Dcom.sun.management.jmxremote.port=<%= @hive_server_jmx_port %>
-Dcom.sun.management.jmxremote.ssl=false
-Dcom.sun.management.jmxremote.authenticate=false"
+ <% end -%>
+fi
\ No newline at end of file
diff --git a/templates/hive/hive-metastore.default.erb
b/templates/hive/hive-metastore.default.erb
deleted file mode 100644
index 0b76249..0000000
--- a/templates/hive/hive-metastore.default.erb
+++ /dev/null
@@ -1,30 +0,0 @@
-# Note: This file is managed by Puppet.
-
-# Licensed to the Apache Software Foundation (ASF) under one or more
-# contributor license agreements. See the NOTICE file distributed with
-# this work for additional information regarding copyright ownership.
-# The ASF licenses this file to You under the Apache License, Version 2.0
-# (the "License"); you may not use this file except in compliance with
-# the License. You may obtain a copy of the License at
-#
-# http://www.apache.org/licenses/LICENSE-2.0
-#
-# Unless required by applicable law or agreed to in writing, software
-# distributed under the License is distributed on an "AS IS" BASIS,
-# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-# See the License for the specific language governing permissions and
-# limitations under the License.
-
-# The port for Hive metastore daemon to listen to.
-# Unfortunatelly, there is no way to specify the interfaces
-# to which the daemon binds.
-#
-<%= @port ? "export PORT=#{@port}" : '#export PORT=' %>
-
-<% if @heapsize -%>
-export HADOOP_OPTS="$HADOOP_OPTS -Xmx<%= @heapsize %>"
-<% end -%>
-
-<% if @jmx_port -%>
-HADOOP_OPTS="$HADOOP_OPTS -Dcom.sun.management.jmxremote.port=<%= @jmx_port %>
-Dcom.sun.management.jmxremote.ssl=false
-Dcom.sun.management.jmxremote.authenticate=false"
-<% end -%>
\ No newline at end of file
diff --git a/templates/hive/hive-server2.default.erb
b/templates/hive/hive-server2.default.erb
deleted file mode 100644
index 9c8e675..0000000
--- a/templates/hive/hive-server2.default.erb
+++ /dev/null
@@ -1,30 +0,0 @@
-# Note: This file is managed by Puppet.
-
-# Licensed to the Apache Software Foundation (ASF) under one or more
-# contributor license agreements. See the NOTICE file distributed with
-# this work for additional information regarding copyright ownership.
-# The ASF licenses this file to You under the Apache License, Version 2.0
-# (the "License"); you may not use this file except in compliance with
-# the License. You may obtain a copy of the License at
-#
-# http://www.apache.org/licenses/LICENSE-2.0
-#
-# Unless required by applicable law or agreed to in writing, software
-# distributed under the License is distributed on an "AS IS" BASIS,
-# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-# See the License for the specific language governing permissions and
-# limitations under the License.
-
-# The port for Hive server2 daemon to listen to.
-# Unfortunatelly, there is no way to specify the interfaces
-# to which the daemon binds.
-#
-<%= @port ? "export PORT=#{@port}" : '#export PORT=' %>
-
-<% if @heapsize -%>
-export HADOOP_OPTS="$HADOOP_OPTS -Xmx<%= @heapsize %>"
-<% end -%>
-
-<% if @jmx_port -%>
-HADOOP_OPTS="$HADOOP_OPTS -Dcom.sun.management.jmxremote.port=<%= @jmx_port %>
-Dcom.sun.management.jmxremote.ssl=false
-Dcom.sun.management.jmxremote.authenticate=false"
-<% end -%>
--
To view, visit https://gerrit.wikimedia.org/r/368820
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I5c0c044ee982843443c6e2c5a2f1fa6ddc949a6f
Gerrit-PatchSet: 2
Gerrit-Project: operations/puppet/cdh
Gerrit-Branch: master
Gerrit-Owner: Elukey <[email protected]>
Gerrit-Reviewer: Elukey <[email protected]>
Gerrit-Reviewer: Ottomata <[email protected]>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
