Skip to content

Commit

Permalink
[KYUUBI #5867][FOLLOWUP] Remove unused config
Browse files Browse the repository at this point in the history 
# 🔍 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 🔖

- [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 ⚰️

#### Behavior With This Pull Request 🎉

#### 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

e4183ff [yikaifei] [KYUUBI #5867][FOLLOWUP] Remove unused config

Authored-by: yikaifei <yikaifei@apache.org>
Signed-off-by: Cheng Pan <chengpan@apache.org>
  • Loading branch information
@yikf @pan3793
yikf authored and pan3793 committed Jan 15, 2024
1 parent 35e988d commit 518e918
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 9 deletions.
1 change: 0 additions & 1 deletion docs/configuration/settings.md
View file
Original file line number Diff line number Diff line change
Expand Up @@ -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. | int | 1.9.0 |
| kyuubi.engine.yarn.priority | &lt;undefined&gt; | kyuubi engine yarn priority when the engine deploy mode is YARN. | int | 1.9.0 |
| kyuubi.engine.yarn.queue | default | kyuubi engine yarn queue when the engine deploy mode is YARN. | string | 1.9.0 |
| kyuubi.engine.yarn.report.interval | PT1S | Interval between reports of the current engine on yarn app status. | duration | 1.9.0 |
| kyuubi.engine.yarn.stagingDir | &lt;undefined&gt; | Staging directory used while submitting kyuubi engine to YARN, It should be a absolute path in HDFS. | string | 1.9.0 |
| kyuubi.engine.yarn.submit.timeout | PT30S | The engine submit timeout for YARN application. | duration | 1.7.2 |
| kyuubi.engine.yarn.tags | &lt;undefined&gt; | kyuubi engine yarn tags when the engine deploy mode is YARN. | seq | 1.9.0 |
Expand Down
8 changes: 0 additions & 8 deletions kyuubi-common/src/main/scala/org/apache/kyuubi/config/KyuubiConf.scala
View file
Original file line number Diff line number Diff line change
Expand Up @@ -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.")
Expand Down

0 comments on commit 518e918

Please sign in to comment.