This is an automated email from the ASF dual-hosted git repository.
chengpan pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/kyuubi.git
The following commit(s) were added to refs/heads/master by this push:
new 518e918fa [KYUUBI #5867][FOLLOWUP] Remove unused config
518e918fa is described below
commit 518e918fa7ded9e46960a5d2cc63f9c364b104c7
Author: yikaifei <[email protected]>
AuthorDate: Mon Jan 15 17:38:18 2024 +0800
[KYUUBI #5867][FOLLOWUP] Remove unused config
# :mag: Description
## Issue References ๐
This pull request aims to remove unused config as #5867 followup.
## Describe Your Solution ๐ง
Please include a summary of the change and which issue is fixed. Please
also include relevant motivation and context. List any dependencies that are
required for this change.
## Types of changes :bookmark:
- [x] Bugfix (non-breaking change which fixes an issue)
- [ ] New feature (non-breaking change which adds functionality)
- [ ] Breaking change (fix or feature that would cause existing
functionality to change)
## Test Plan ๐งช
#### Behavior Without This Pull Request :coffin:
#### Behavior With This Pull Request :tada:
#### Related Unit Tests
---
# Checklist ๐
- [x] This patch was not authored or co-authored using [Generative
Tooling](https://www.apache.org/legal/generative-tooling.html)
**Be nice. Be informative.**
Closes #5976 from Yikf/remove-INTERVAL.
Closes #5867
e4183ffca [yikaifei] [KYUUBI #5867][FOLLOWUP] Remove unused config
Authored-by: yikaifei <[email protected]>
Signed-off-by: Cheng Pan <[email protected]>
---
docs/configuration/settings.md | 1 -
.../src/main/scala/org/apache/kyuubi/config/KyuubiConf.scala | 8 --------
2 files changed, 9 deletions(-)
diff --git a/docs/configuration/settings.md b/docs/configuration/settings.md
index db6edbf2b..74485f7ff 100644
--- a/docs/configuration/settings.md
+++ b/docs/configuration/settings.md
@@ -211,7 +211,6 @@ You can configure the Kyuubi properties in
`$KYUUBI_HOME/conf/kyuubi-defaults.co
| kyuubi.engine.yarn.memory | 1024
| kyuubi engine container memory in mb when the engine deploy mode is
YARN.
[...]
| kyuubi.engine.yarn.priority | <undefined>
| kyuubi engine yarn priority when the engine deploy mode is YARN.
[...]
| kyuubi.engine.yarn.queue | default
| kyuubi engine yarn queue when the engine deploy mode is YARN.
[...]
-| kyuubi.engine.yarn.report.interval | PT1S
| Interval between reports of the current engine on yarn app status.
[...]
| kyuubi.engine.yarn.stagingDir | <undefined>
| Staging directory used while submitting kyuubi engine to YARN, It
should be a absolute path in HDFS.
[...]
| kyuubi.engine.yarn.submit.timeout | PT30S
| The engine submit timeout for YARN application.
[...]
| kyuubi.engine.yarn.tags | <undefined>
| kyuubi engine yarn tags when the engine deploy mode is YARN.
[...]
diff --git
a/kyuubi-common/src/main/scala/org/apache/kyuubi/config/KyuubiConf.scala
b/kyuubi-common/src/main/scala/org/apache/kyuubi/config/KyuubiConf.scala
index 926776209..cdf6db3e5 100644
--- a/kyuubi-common/src/main/scala/org/apache/kyuubi/config/KyuubiConf.scala
+++ b/kyuubi-common/src/main/scala/org/apache/kyuubi/config/KyuubiConf.scala
@@ -2708,14 +2708,6 @@ object KyuubiConf {
.stringConf
.createOptional
- val ENGINE_DEPLOY_YARN_MODE_REPORT_INTERVAL: ConfigEntry[Long] =
- buildConf("kyuubi.engine.yarn.report.interval")
- .doc("Interval between reports of the current engine on yarn app
status.")
- .version("1.9.0")
- .timeConf
- .checkValue(t => t > 0, "must be positive integer")
- .createWithDefault(Duration.ofSeconds(1).toMillis)
-
val ENGINE_DEPLOY_YARN_MODE_TAGS: OptionalConfigEntry[Seq[String]] =
buildConf("kyuubi.engine.yarn.tags")
.doc(s"kyuubi engine yarn tags when the engine deploy mode is YARN.")
