You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@dolphinscheduler.apache.org by "ruanwenjun (via GitHub)" <gi...@apache.org> on 2023/07/26 13:26:01 UTC
[GitHub] [dolphinscheduler] ruanwenjun commented on issue #14495: [Bug] [DolphinScheduler Standalone-Server] MasterServer remove registry path exception
ruanwenjun commented on issue #14495:
URL: https://github.com/apache/dolphinscheduler/issues/14495#issuecomment-1651803392
>  When you stop the standalone-server, program try to remove master node in zookeeper.At this point zookeeper's state is **stopped** . So, the program throw the exception. @fincentxyz
>
> Considering that the master node was created by EPHEMERAL mode, should we remove this line in the deregister() method (MasterRegistryClient.java)? `registryClient.remove(masterConfig.getMasterRegistryPath());` @ruanwenjun
Yes, it's of to remove this code, but not all registry plugin is implemented as EPHEMERAL, so keep the remove logic is ok, but it shouldn't throw exception if the node not exist.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@dolphinscheduler.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org