Skip to content

Commit

Permalink
[KYUUBI #5447][FOLLOWUP] Remove unrelated debug prints in TableIdenti…
Browse files Browse the repository at this point in the history
…fierTableExtractor

### _Why are the changes needed?_
To close #5447
Remove unrelated debug change in  tableExtractor

### _How was this patch tested?_
- [ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible

- [ ] Add screenshots for manual tests if appropriate

- [x] [Run test](https://kyuubi.readthedocs.io/en/master/contributing/code/testing.html#running-tests) locally before make a pull request

### _Was this patch authored or co-authored using generative AI tooling?_
No

Closes #5494 from AngersZhuuuu/KYUUBI-5447-FOLLOWUP.

Closes #5447

90a0bcd [Bowen Liang] Update extensions/spark/kyuubi-spark-authz/src/main/scala/org/apache/kyuubi/plugin/spark/authz/serde/tableExtractors.scala
85941e2 [Angerszhuuuu] [KYUUBI #5447][AUTHZ] Remove unrelated debug change in  tableExtractor

Lead-authored-by: Angerszhuuuu <[email protected]>
Co-authored-by: Bowen Liang <[email protected]>
Signed-off-by: Cheng Pan <[email protected]>
  • Loading branch information
2 people authored and pan3793 committed Oct 22, 2023
1 parent abaa369 commit 03d6223
Showing 1 changed file with 1 addition and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -80,9 +80,7 @@ class TableIdentifierTableExtractor extends TableExtractor {
val catalogTable = spark.sessionState.catalog.getTableMetadata(identifier)
Option(catalogTable.owner).filter(_.nonEmpty)
} catch {
case e: Exception =>
e.printStackTrace()
None
case _: Exception => None
}
Some(Table(None, identifier.database, identifier.table, owner))
}
Expand Down

0 comments on commit 03d6223

Please sign in to comment.