You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kyuubi.apache.org by bo...@apache.org on 2023/02/06 07:50:06 UTC
[kyuubi] branch master updated: [KYUUBI #4209] [Authz] Set logging level to DEBUG in PrivilegesBuilder for multiple specs
This is an automated email from the ASF dual-hosted git repository.
bowenliang 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 534fc9f20 [KYUUBI #4209] [Authz] Set logging level to DEBUG in PrivilegesBuilder for multiple specs
534fc9f20 is described below
commit 534fc9f20f864069b76f40c3e5b0e183bee8dd62
Author: liangbowen <li...@gf.com.cn>
AuthorDate: Mon Feb 6 15:49:56 2023 +0800
[KYUUBI #4209] [Authz] Set logging level to DEBUG in PrivilegesBuilder for multiple specs
### _Why are the changes needed?_
- change logging level from `warn` to `debug` in Authz
- to eliminate duplicated unuseful messages (as screenshot below) with multiple specs in extracting resources with lower Spark versions
![WechatIMG7517](https://user-images.githubusercontent.com/1935105/215329357-ad242cf4-9c98-44b0-978a-f37304a85ef0.png)
### _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
- [ ] [Run test](https://kyuubi.readthedocs.io/en/master/develop_tools/testing.html#running-tests) locally before make a pull request
Closes #4209 from bowenliang123/authz-debug.
Closes #4209
f3b949bc [liangbowen] change logging level from `warn` to `debug` in multi extractor rule in Authz serde
Authored-by: liangbowen <li...@gf.com.cn>
Signed-off-by: liangbowen <li...@gf.com.cn>
---
.../org/apache/kyuubi/plugin/spark/authz/PrivilegesBuilder.scala | 6 +++---
.../org/apache/kyuubi/plugin/spark/authz/serde/CommandSpec.scala | 4 ++--
2 files changed, 5 insertions(+), 5 deletions(-)
diff --git a/extensions/spark/kyuubi-spark-authz/src/main/scala/org/apache/kyuubi/plugin/spark/authz/PrivilegesBuilder.scala b/extensions/spark/kyuubi-spark-authz/src/main/scala/org/apache/kyuubi/plugin/spark/authz/PrivilegesBuilder.scala
index cee79b87d..51f5694e1 100644
--- a/extensions/spark/kyuubi-spark-authz/src/main/scala/org/apache/kyuubi/plugin/spark/authz/PrivilegesBuilder.scala
+++ b/extensions/spark/kyuubi-spark-authz/src/main/scala/org/apache/kyuubi/plugin/spark/authz/PrivilegesBuilder.scala
@@ -144,7 +144,7 @@ object PrivilegesBuilder {
}
} catch {
case e: Exception =>
- LOG.warn(tableDesc.error(plan, e))
+ LOG.debug(tableDesc.error(plan, e))
Nil
}
}
@@ -162,7 +162,7 @@ object PrivilegesBuilder {
}
} catch {
case e: Exception =>
- LOG.warn(databaseDesc.error(plan, e))
+ LOG.debug(databaseDesc.error(plan, e))
}
}
desc.operationType
@@ -193,7 +193,7 @@ object PrivilegesBuilder {
}
} catch {
case e: Exception =>
- LOG.warn(fd.error(plan, e))
+ LOG.debug(fd.error(plan, e))
}
}
spec.operationType
diff --git a/extensions/spark/kyuubi-spark-authz/src/main/scala/org/apache/kyuubi/plugin/spark/authz/serde/CommandSpec.scala b/extensions/spark/kyuubi-spark-authz/src/main/scala/org/apache/kyuubi/plugin/spark/authz/serde/CommandSpec.scala
index d72d78932..e96ef8cbf 100644
--- a/extensions/spark/kyuubi-spark-authz/src/main/scala/org/apache/kyuubi/plugin/spark/authz/serde/CommandSpec.scala
+++ b/extensions/spark/kyuubi-spark-authz/src/main/scala/org/apache/kyuubi/plugin/spark/authz/serde/CommandSpec.scala
@@ -85,7 +85,7 @@ case class TableCommandSpec(
qd.extract(plan)
} catch {
case e: Exception =>
- LOG.warn(qd.error(plan, e))
+ LOG.debug(qd.error(plan, e))
None
}
}
@@ -102,7 +102,7 @@ case class ScanSpec(
td.extract(plan, spark)
} catch {
case e: Exception =>
- LOG.warn(td.error(plan, e))
+ LOG.debug(td.error(plan, e))
None
}
}