From 03b7112eee5a87da059e22716ea54598e5b76bed Mon Sep 17 00:00:00 2001 From: ulysses-you Date: Mon, 8 Mar 2021 11:22:19 +0800 Subject: [PATCH] [KYUUBI #403] FIx flaky test in ConnectionLevelSparkEngineSuite ![ulysses-you](https://badgen.net/badge/Hello/ulysses-you/green) [![Closes #403](https://badgen.net/badge/Preview/Closes%20%23403/blue)](https://github.com/yaooqinn/kyuubi/pull/403) ![1](https://badgen.net/badge/%2B/1/red) ![1](https://badgen.net/badge/-/1/green) ![4](https://badgen.net/badge/commits/4/yellow) ![Target Issue](https://badgen.net/badge/Missing/Target%20Issue/ff0000) ![Test Plan](https://badgen.net/badge/Missing/Test%20Plan/ff0000) [❨?❩](https://pullrequestbadge.com/?utm_medium=github&utm_source=yaooqinn&utm_campaign=badge_info) ### _Why are the changes needed?_ ### _How was this patch tested?_ `ignore` -> `test` Closes #403 from ulysses-you/flaky-test2. d8aa6c7 [ulysses-you] empty fd38e9c [ulysses-you] empty 2e5a7ee [ulysses-you] empty 444a526 [ulysses-you] fix Authored-by: ulysses-you Signed-off-by: ulysses-you --- .../apache/kyuubi/engine/spark/ShareLevelSparkEngineSuite.scala | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/externals/kyuubi-spark-sql-engine/src/test/scala/org/apache/kyuubi/engine/spark/ShareLevelSparkEngineSuite.scala b/externals/kyuubi-spark-sql-engine/src/test/scala/org/apache/kyuubi/engine/spark/ShareLevelSparkEngineSuite.scala index ea1777e53b4..c6ce8c4b55c 100644 --- a/externals/kyuubi-spark-sql-engine/src/test/scala/org/apache/kyuubi/engine/spark/ShareLevelSparkEngineSuite.scala +++ b/externals/kyuubi-spark-sql-engine/src/test/scala/org/apache/kyuubi/engine/spark/ShareLevelSparkEngineSuite.scala @@ -41,7 +41,7 @@ abstract class ShareLevelSparkEngineSuite s"/kyuubi/${sharedLevel.toString}/${UUID.randomUUID().toString}" } - ignore("check discovery service is clean up with different share level") { + test("check discovery service is clean up with different share level") { withZkClient { zkClient => assert(engine.getServiceState == ServiceState.STARTED) assert(zkClient.checkExists().forPath(namespace) != null)