From ecd1c0e1a64c3161ac2dab9d33d2026f5fdebd22 Mon Sep 17 00:00:00 2001
From: jhx1008 <jhx1008@gmail.com>
Date: Sat, 30 Jan 2021 13:36:45 +0800
Subject: [PATCH] [KYUUBI #323] Fix Kyuubi conf template

![yaooqinn](https://badgen.net/badge/Hello/yaooqinn/green) [![Closes%20#323](https://badgen.net/badge/Preview/Closes%2520%23323/blue)](https://github.com/yaooqinn/kyuubi/pull/323) ![0](https://badgen.net/badge/%2B/0/red) ![10](https://badgen.net/badge/-/10/green) ![2](https://badgen.net/badge/commits/2/yellow) ![Test Plan](https://badgen.net/badge/Missing/Test%20Plan/ff0000) [&#10088;?&#10089;](https://pullrequestbadge.com/?utm_medium=github&utm_source=yaooqinn&utm_campaign=badge_info)<!-- PR-BADGE: PLEASE DO NOT REMOVE THIS COMMENT -->

<!--
Thanks for sending a pull request!

Here are some tips for you:
  1. If this is your first time, please read our contributor guidelines:
     https://kyuubi.readthedocs.io/en/latest/community/contributions.html
-->

### _Which issue are you going to fix?_
<!--
Replace ${ID} below with the actual issue id from
https://github.com/yaooqinn/kyuubi/issues,
so that the issue will be linked and automatically closed after merging
-->

Fixes #${ID}

### _Why are the changes needed?_
<!--
Please clarify why the changes are needed. For instance,
  1. If you add a feature, you can talk about the user case of it.
  2. If you fix a bug, you can clarify why it is a bug.
-->

the Hadoop conf can not be recognized in `kyuubi-defaulfs.conf`, use HADOOP_CONF_DIR instead

### _How was this patch tested?_
- [ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible

- [ ] Add screenshots for manual tests if appropriate

- [ ] [Run test](https://kyuubi.readthedocs.io/en/latest/tools/testing.html#running-tests) locally before make a pull request

Closes #323 from yaooqinn/conf.

a5f8a66 [Kent Yao] nit
7ef0046 [jhx1008] Fix Kyuubi conf template

Lead-authored-by: jhx1008 <jhx1008@gmail.com>
Co-authored-by: Kent Yao <yao@apache.org>
Signed-off-by: Kent Yao <yao@apache.org>
---
 conf/kyuubi-defaults.conf.template | 5 -----
 docs/deployment/settings.md        | 5 -----
 2 files changed, 10 deletions(-)

diff --git a/conf/kyuubi-defaults.conf.template b/conf/kyuubi-defaults.conf.template
index b70f6eaf4..91bacb3ea 100644
--- a/conf/kyuubi-defaults.conf.template
+++ b/conf/kyuubi-defaults.conf.template
@@ -165,8 +165,3 @@
 # spark.sql.adaptive.skewJoin.skewedPartitionFactor         5
 # spark.sql.adaptive.skewJoin.skewedPartitionThresholdInBytes 256m
 # spark.sql.adaptive.nonEmptyPartitionRatioForBroadcastJoin 0.2
-
-## Hadoop Configurations, they will override those in $HADOOP_CONF_DIR
-#
-# hadoop.security.authentication  KERBEROS
-#
diff --git a/docs/deployment/settings.md b/docs/deployment/settings.md
index 2a06632f6..a0d83d136 100644
--- a/docs/deployment/settings.md
+++ b/docs/deployment/settings.md
@@ -238,11 +238,6 @@ You can configure the Kyuubi properties in `$KYUUBI_HOME/conf/kyuubi-defaults.co
 # spark.sql.adaptive.skewJoin.skewedPartitionFactor         5
 # spark.sql.adaptive.skewJoin.skewedPartitionThresholdInBytes 256m
 # spark.sql.adaptive.nonEmptyPartitionRatioForBroadcastJoin 0.2
-
-## Hadoop Configurations, they will override those in $HADOOP_CONF_DIR
-#
-# hadoop.security.authentication  KERBEROS
-#
 ```
 ### Authentication