wen-hemin commented on a change in pull request #5814:
URL: https://github.com/apache/dolphinscheduler/pull/5814#discussion_r669245184
##########
File path:
dolphinscheduler-api/src/main/java/org/apache/dolphinscheduler/api/utils/RegistryCenterUtils.java
##########
@@ -36,21 +36,14 @@
* fixme Some of the information obtained in the api belongs to the unique
information of zk.
* I am not sure whether there is a good abstraction method. This is related
to whether the specific plug-in is provided.
*/
-@Component
-public class RegistryMonitor {
+public class RegistryCenterUtils {
- @Autowired
- RegistryClient registryClient;
-
- @PostConstruct
- public void initRegistry() {
- registryClient.init();
- }
+ private static RegistryClient registryClient =
RegistryClient.getInstance();
/**
* @return zookeeper info list
*/
- public List<ZookeeperRecord> zookeeperInfoList() {
+ public static List<ZookeeperRecord> zookeeperInfoList() {
Review comment:
After this method is confirmed, it may be deleted together with the
frontend in the future.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]