Skip to content

Commit

Permalink
[KYUUBI apache#5985] [AUTHZ][MINOR] Remove incorrect getUri method
Browse files Browse the repository at this point in the history
# 🔍 Description
## Issue References 🔗

This pull request remove incorrect getUri method in authZ module, This method is currently not applicable in any context, and it is incorrect as it ought to return a List type rather than a String.

## 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 apache#5985 from Yikf/remove-incorrect-getUrl.

Closes apache#5985

93ee549 [yikaifei] remove incorrect getUri

Authored-by: yikaifei <[email protected]>
Signed-off-by: Cheng Pan <[email protected]>
  • Loading branch information
yikf authored and zhaohehuhu committed Mar 21, 2024
1 parent 75c1c07 commit aefed84
Showing 1 changed file with 0 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,6 @@ class AccessResource private (val objectType: ObjectType, val catalog: Option[St
val columnStr = getColumn
if (columnStr == null) Nil else columnStr.split(",").filter(_.nonEmpty)
}
def getUrl: String = getValue("url")
}

object AccessResource {
Expand Down

0 comments on commit aefed84

Please sign in to comment.