Skip to content

Commit

Permalink
[KYUUBI #5991] Error on reading Atlas properties composed of multi va…
Browse files Browse the repository at this point in the history
…lues

# 🔍 Description
## Issue References 🔗

This pull request fixes #

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

- [ ] This patch was not authored or co-authored using [Generative Tooling](https://www.apache.org/legal/generative-tooling.html)

**Be nice. Be informative.**

Closes #5993 from SwordyZhao/fix-issues-5991.

Closes #5991

827007d [swordy_zhao] run dev/reformat fix code style.
600363d [swordy_zhao] delete scala.List,Convert a java.List to scala.List
7b000e9 [swordy_zhao] fix 5991--kyuubi failed to read atlas.rest.address
5de0576 [swordy_zhao] fix 5991--kyuubi failed to read atlas.rest.address

Authored-by: swordy_zhao <swordy_work@163.com>
Signed-off-by: Cheng Pan <chengpan@apache.org>
  • Loading branch information
SwordyZhao authored and pan3793 committed Feb 6, 2024
1 parent 5ef4390 commit 97f7987
Showing 1 changed file with 5 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,10 @@

package org.apache.kyuubi.plugin.lineage.dispatcher.atlas

import java.util

import scala.collection.JavaConverters._

import org.apache.atlas.ApplicationProperties
import org.apache.commons.configuration.Configuration
import org.apache.spark.kyuubi.lineage.SparkContextHelper
Expand All @@ -26,7 +30,7 @@ class AtlasClientConf(configuration: Configuration) {
def get(entry: ConfigEntry): String = {
configuration.getProperty(entry.key) match {
case s: String => s
case l: List[_] => l.mkString(",")
case jl: util.List[_] => jl.asScala.mkString(",")
case o if o != null => o.toString
case _ => entry.defaultValue
}
Expand Down

0 comments on commit 97f7987

Please sign in to comment.