This is an automated email from the ASF dual-hosted git repository.
chengpan pushed a commit to branch branch-1.7
in repository https://gitbox.apache.org/repos/asf/kyuubi.git
The following commit(s) were added to refs/heads/branch-1.7 by this push:
new 467c044ab [KYUUBI #4985] [TEST] Fix loglevel restore behavior of
`KyuubiFunSuite#withLogAppender`
467c044ab is described below
commit 467c044aba14bd66379879d67006350c9ba6cdef
Author: Cheng Pan <[email protected]>
AuthorDate: Sun Jun 25 10:42:41 2023 +0800
[KYUUBI #4985] [TEST] Fix loglevel restore behavior of
`KyuubiFunSuite#withLogAppender`
### _Why are the changes needed?_
This method is forked from Apache Spark, and recently,
[SPARK-44074](https://github.com/apache/spark/issues/41663) identified and
fixed the issue about loglevel restore behavior.
### _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
Closes #4985 from pan3793/test-log.
Closes #4985
4c05bc522 [Cheng Pan] Fix loglevel restore behavior of
`KyuubiFunSuite#withLogAppender`
Authored-by: Cheng Pan <[email protected]>
Signed-off-by: Cheng Pan <[email protected]>
(cherry picked from commit a622c0a8e88b7f39400b66f153ac8176c37a16f7)
Signed-off-by: Cheng Pan <[email protected]>
---
kyuubi-common/src/test/scala/org/apache/kyuubi/KyuubiFunSuite.scala | 1 +
1 file changed, 1 insertion(+)
diff --git
a/kyuubi-common/src/test/scala/org/apache/kyuubi/KyuubiFunSuite.scala
b/kyuubi-common/src/test/scala/org/apache/kyuubi/KyuubiFunSuite.scala
index 96a612aab..2789d7f89 100644
--- a/kyuubi-common/src/test/scala/org/apache/kyuubi/KyuubiFunSuite.scala
+++ b/kyuubi-common/src/test/scala/org/apache/kyuubi/KyuubiFunSuite.scala
@@ -102,6 +102,7 @@ trait KyuubiFunSuite extends AnyFunSuite
logger.asInstanceOf[Logger].setLevel(restoreLevels(i))
logger.asInstanceOf[Logger].get().setLevel(restoreLevels(i))
}
+
LogManager.getContext(false).asInstanceOf[LoggerContext].updateLoggers()
}
}
}
