[KYUUBI #2924] Correct the frontend server start state

### _Why are the changes needed?_

CI has a lot of flaky test, it is because we will mark server is started when we call thread.start, but the server (bianry or http) may not be work at that moment. As a result, the client can not connect a started server.

### _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 #2925 from ulysses-you/correct-start-state.

Closes #2924

b5a30696 [ulysses-you] test
bb4b4a8a [ulysses-you] sleep

Authored-by: ulysses-you <ulyssesyou18@gmail.com>
Signed-off-by: ulysses-you <ulyssesyou@apache.org>
This commit is contained in:
ulysses-you 2022-06-27 09:51:22 +08:00 committed by ulysses-you
parent 9bc0aa6764
commit de2e11c2bc
No known key found for this signature in database
GPG Key ID: 4C500BC62D576766
2 changed files with 10 additions and 0 deletions

View File

@ -27,6 +27,11 @@ import org.apache.kyuubi.service.authentication.UserDefineAuthenticationProvider
class KyuubiOperationThriftHttpKerberosAndPlainAuthSuite
extends KyuubiOperationKerberosAndPlainAuthSuite {
override def beforeAll(): Unit = {
super.beforeAll()
Thread.sleep(3000)
}
override protected val frontendProtocols: Seq[KyuubiConf.FrontendProtocols.Value] =
FrontendProtocols.THRIFT_HTTP :: Nil

View File

@ -22,6 +22,11 @@ import org.apache.kyuubi.config.KyuubiConf.FrontendProtocols
import org.apache.kyuubi.operation.KyuubiOperationPerUserSuite
class KyuubiOperationThriftHttpPerUserSuite extends KyuubiOperationPerUserSuite {
override def beforeAll(): Unit = {
super.beforeAll()
Thread.sleep(3000)
}
override protected val frontendProtocols: Seq[KyuubiConf.FrontendProtocols.Value] =
FrontendProtocols.THRIFT_HTTP :: Nil