From af3f858f3a93c949d8d582aec7cfde9d8d936b1c Mon Sep 17 00:00:00 2001 From: Min Zhao Date: Sun, 4 Jul 2021 02:06:40 +0800 Subject: [PATCH] [KYUUBI #744] Log update ### _Why are the changes needed?_ 1. add '=' in start of line to keep the consistent code style. 2. update log info to correct position. ### _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/latest/tools/testing.html#running-tests) locally before make a pull request Closes #744 from zhaomin1423/log_update. Closes #744 bf1e0d8a [Min Zhao] update log 'Closing ...' to corrent position and add 'Closed...' 6907430f [Min Zhao] update log info to keep the consistent code style Authored-by: Min Zhao Signed-off-by: Kent Yao --- .../src/test/scala/org/apache/kyuubi/ThreadAudit.scala | 2 +- .../test/scala/org/apache/kyuubi/operation/JDBCTestUtils.scala | 3 ++- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/kyuubi-common/src/test/scala/org/apache/kyuubi/ThreadAudit.scala b/kyuubi-common/src/test/scala/org/apache/kyuubi/ThreadAudit.scala index 08fffb9e7..3f7ace6fe 100644 --- a/kyuubi-common/src/test/scala/org/apache/kyuubi/ThreadAudit.scala +++ b/kyuubi-common/src/test/scala/org/apache/kyuubi/ThreadAudit.scala @@ -36,7 +36,7 @@ trait ThreadAudit extends Logging { s"thread names: ${remainingThreadNames.mkString(", ")} =====\n") } } else { - warn(s"\n\n= THREAD AUDIT POST ACTION CALLED WITHOUT PRE ACTION IN SUITE $shortName ==\n") + warn(s"\n\n== THREAD AUDIT POST ACTION CALLED WITHOUT PRE ACTION IN SUITE $shortName ==\n") } } diff --git a/kyuubi-common/src/test/scala/org/apache/kyuubi/operation/JDBCTestUtils.scala b/kyuubi-common/src/test/scala/org/apache/kyuubi/operation/JDBCTestUtils.scala index 705364a4b..cd1cfe571 100644 --- a/kyuubi-common/src/test/scala/org/apache/kyuubi/operation/JDBCTestUtils.scala +++ b/kyuubi-common/src/test/scala/org/apache/kyuubi/operation/JDBCTestUtils.scala @@ -79,8 +79,9 @@ trait JDBCTestUtils extends KyuubiFunSuite { info("Closing statements") statements.foreach(_.close()) info("Closed statements") - connections.foreach(_.close()) info("Closing connections") + connections.foreach(_.close()) + info("Closed connections") } }