Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Bug] Kyuubi server not stop when failed to initialize frontend service #1449

Closed
3 tasks done
zhenjiaguo opened this issue Nov 24, 2021 · 1 comment
Closed
3 tasks done

Comments

@zhenjiaguo
Copy link
Contributor

Code of Conduct

Search before asking

  • I have searched in the issues and found no similar issues.

Describe the bug

When I config like this:

kyuubi.authentication=Non

Kyuubi server fail to start but not stop. Log:

2021-11-24 21:43:45.332 WARN kyuubi.Utils: jia-pc was resolved to a loopback address: 127.0.1.1, using 192.168.83.190
Exception in thread "main" org.apache.kyuubi.KyuubiException: Failed to initialize frontend service on /192.168.83.190:10009.
        at org.apache.kyuubi.service.ThriftBinaryFrontendService.initialize(ThriftBinaryFrontendService.scala:110)
        at org.apache.kyuubi.service.CompositeService.$anonfun$initialize$1(CompositeService.scala:40)
        at org.apache.kyuubi.service.CompositeService.$anonfun$initialize$1$adapted(CompositeService.scala:40)
        at scala.collection.mutable.ResizableArray.foreach(ResizableArray.scala:62)
        at scala.collection.mutable.ResizableArray.foreach$(ResizableArray.scala:55)
        at scala.collection.mutable.ArrayBuffer.foreach(ArrayBuffer.scala:49)
        at org.apache.kyuubi.service.CompositeService.initialize(CompositeService.scala:40)
        at org.apache.kyuubi.service.Serverable.initialize(Serverable.scala:46)
        at org.apache.kyuubi.server.KyuubiServer.initialize(KyuubiServer.scala:155)
        at org.apache.kyuubi.server.KyuubiServer$.startServer(KyuubiServer.scala:93)
        at org.apache.kyuubi.server.KyuubiServer$.main(KyuubiServer.scala:122)
        at org.apache.kyuubi.server.KyuubiServer.main(KyuubiServer.scala)
Caused by: java.lang.IllegalArgumentException: the authentication type should be one or more of NOSASL,NONE,LDAP,KERBEROS,CUSTOM
        at org.apache.kyuubi.config.TypedConfigBuilder.$anonfun$checkValue$1(ConfigBuilder.scala:139)
        at org.apache.kyuubi.config.TypedConfigBuilder.$anonfun$transform$1(ConfigBuilder.scala:134)
        at scala.Option.map(Option.scala:230)
        at org.apache.kyuubi.config.ConfigEntryWithDefault.readFrom(ConfigEntry.scala:96)
        at org.apache.kyuubi.config.KyuubiConf.get(KyuubiConf.scala:87)
        at org.apache.kyuubi.service.authentication.KyuubiAuthenticationFactory.<init>(KyuubiAuthenticationFactory.scala:39)
        at org.apache.kyuubi.service.ThriftBinaryFrontendService.initialize(ThriftBinaryFrontendService.scala:80)
        ... 11 more

Shell:

$ ps -ef | grep kyuubi
jia       385782    1946  0 21:43 pts/8    00:00:09 /usr/lib/jvm/java-1.8.0-openjdk-amd64/bin/java -cp /home/jia/work/apache-kyuubi-1.4.0-SNAPSHOT-bin-2021-11-24/jars/*:/home/jia/work/apache-kyuubi-1.4.0-SNAPSHOT-bin-2021-11-24/conf: org.apache.kyuubi.server.KyuubiServer
jia       387432  384429  0 22:03 pts/8    00:00:00 vim /home/jia/work/apache-kyuubi-1.4.0-SNAPSHOT-bin-2021-11-24/logs/kyuubi-jia-org.apache.kyuubi.server.KyuubiServer-jia-pc.out
jia       388911  301567  0 22:19 pts/2    00:00:00 grep --color=auto kyuubi
$ lsof -i:10009
$ 

Affects Version(s)

master

Kyuubi Server Log Output

No response

Kyuubi Engine Log Output

No response

Kyuubi Server Configurations

No response

Kyuubi Engine Configurations

No response

Additional context

No response

Are you willing to submit PR?

  • Yes I am willing to submit a PR!
@zhenjiaguo zhenjiaguo added the bug label Nov 24, 2021
yaooqinn pushed a commit that referenced this issue Dec 1, 2021
<!--
Thanks for sending a pull request!

Here are some tips for you:
  1. If this is your first time, please read our contributor guidelines: https://kyuubi.readthedocs.io/en/latest/community/contributions.html
  2. If the PR is related to an issue in https://github.com/apache/incubator-kyuubi/issues, add '[KYUUBI #XXXX]' in your PR title, e.g., '[KYUUBI #XXXX] Your PR title ...'.
  3. If the PR is unfinished, add '[WIP]' in your PR title, e.g., '[WIP][KYUUBI #XXXX] Your PR title ...'.
-->

### _Why are the changes needed?_
<!--
Please clarify why the changes are needed. For instance,
  1. If you add a feature, you can talk about the use case of it.
  2. If you fix a bug, you can clarify why it is a bug.
-->

Kyuubi server should exit when initialize fail.

Detail see #1449.

### _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/latest/develop_tools/testing.html#running-tests) locally before make a pull request

Closes #1468 from zhenjiaguo/fix-TBFS-init-fail-not-exit.

Closes #1449

4467a24 [zhenjiaguo] add status judgment before zkServer stop
4208663 [zhenjiaguo] stop zkServer when kyuubi server init fail
1ff3a82 [zhenjiaguo] try catch server initialize

Authored-by: zhenjiaguo <zhenjia_guo@163.com>
Signed-off-by: Kent Yao <yao@apache.org>
(cherry picked from commit 5a4e370)
Signed-off-by: Kent Yao <yao@apache.org>
pan3793 added a commit to pan3793/kyuubi that referenced this issue Dec 13, 2021
pan3793 added a commit that referenced this issue Dec 13, 2021
### _Why are the changes needed?_

Fix import to pass CI

### _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/latest/develop_tools/testing.html#running-tests) locally before make a pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants
@zhenjiaguo and others