(kyuubi) branch master updated: [KYUUBI #5985] [AUTHZ][MINOR] Remove incorrect getUri method

[ch...@apache.org]

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 e9e2d189b [KYUUBI #5985] [AUTHZ][MINOR] Remove incorrect getUri method
e9e2d189b is described below

commit e9e2d189ba05c899fd85041e2a3bb7db4f5f009f
Author: yikaifei <yi...@apache.org>
AuthorDate: Wed Jan 17 09:50:53 2024 +0800

    [KYUUBI #5985] [AUTHZ][MINOR] Remove incorrect getUri method
    
    # :mag: 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 :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 #5985 from Yikf/remove-incorrect-getUrl.
    
    Closes #5985
    
    93ee5498e [yikaifei] remove incorrect getUri
    
    Authored-by: yikaifei <yi...@apache.org>
    Signed-off-by: Cheng Pan <ch...@apache.org>
---
 .../org/apache/kyuubi/plugin/spark/authz/ranger/AccessResource.scala     | 1 -
 1 file changed, 1 deletion(-)

diff --git a/extensions/spark/kyuubi-spark-authz/src/main/scala/org/apache/kyuubi/plugin/spark/authz/ranger/AccessResource.scala b/extensions/spark/kyuubi-spark-authz/src/main/scala/org/apache/kyuubi/plugin/spark/authz/ranger/AccessResource.scala
index 858dc1c37..7772c86b7 100644
--- a/extensions/spark/kyuubi-spark-authz/src/main/scala/org/apache/kyuubi/plugin/spark/authz/ranger/AccessResource.scala
+++ b/extensions/spark/kyuubi-spark-authz/src/main/scala/org/apache/kyuubi/plugin/spark/authz/ranger/AccessResource.scala
@@ -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 {