From 0c53d00799624e8e721b2ef9fcaf53d41005cf43 Mon Sep 17 00:00:00 2001 From: Angerszhuuuu Date: Sun, 22 Oct 2023 22:35:39 +0800 Subject: [PATCH] [KYUUBI #5447][FOLLOWUP] Remove unrelated debug prints in TableIdentifierTableExtractor ### _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 90a0bcd91 [Bowen Liang] Update extensions/spark/kyuubi-spark-authz/src/main/scala/org/apache/kyuubi/plugin/spark/authz/serde/tableExtractors.scala 85941e26c [Angerszhuuuu] [KYUUBI #5447][AUTHZ] Remove unrelated debug change in tableExtractor Lead-authored-by: Angerszhuuuu Co-authored-by: Bowen Liang Signed-off-by: Cheng Pan --- .../kyuubi/plugin/spark/authz/serde/tableExtractors.scala | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/extensions/spark/kyuubi-spark-authz/src/main/scala/org/apache/kyuubi/plugin/spark/authz/serde/tableExtractors.scala b/extensions/spark/kyuubi-spark-authz/src/main/scala/org/apache/kyuubi/plugin/spark/authz/serde/tableExtractors.scala index 47c486af360..fcd8c8207e7 100644 --- a/extensions/spark/kyuubi-spark-authz/src/main/scala/org/apache/kyuubi/plugin/spark/authz/serde/tableExtractors.scala +++ b/extensions/spark/kyuubi-spark-authz/src/main/scala/org/apache/kyuubi/plugin/spark/authz/serde/tableExtractors.scala @@ -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)) }