From 45531f01f5f74283aac266bda026414db8d01c28 Mon Sep 17 00:00:00 2001 From: ulysses-you Date: Fri, 20 May 2022 16:03:08 +0800 Subject: [PATCH] [KYUUBI #2686][FOLLOWUP] Avoid potential flaky test ### _Why are the changes needed?_ increase the time span of the lock timout for test ### _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.apache.org/docs/latest/develop_tools/testing.html#running-tests) locally before make a pull request Closes #2711 from ulysses-you/KYUUBI-2686-FOLLOWUP. Closes #2686 f34e4e22 [ulysses-you] avoid potential flaky test Authored-by: ulysses-you Signed-off-by: Fei Wang --- .../src/main/scala/org/apache/kyuubi/engine/EngineRef.scala | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/kyuubi-server/src/main/scala/org/apache/kyuubi/engine/EngineRef.scala b/kyuubi-server/src/main/scala/org/apache/kyuubi/engine/EngineRef.scala index cfeb0aaf379..6a102afe263 100644 --- a/kyuubi-server/src/main/scala/org/apache/kyuubi/engine/EngineRef.scala +++ b/kyuubi-server/src/main/scala/org/apache/kyuubi/engine/EngineRef.scala @@ -73,7 +73,7 @@ private[kyuubi] class EngineRef( // In case the multi kyuubi instances have the small gap of timeout, here we add // a small amount of time for timeout - private val LOCK_TIMEOUT_SPAN_FACTOR = 0.1 + private val LOCK_TIMEOUT_SPAN_FACTOR = if (Utils.isTesting) 0.5 else 0.1 private var builder: ProcBuilder = _