You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@dolphinscheduler.apache.org by GitBox <gi...@apache.org> on 2022/01/06 14:22:19 UTC
[GitHub] [dolphinscheduler-website] aaronlinv opened a new pull request #620: Remove logger server
aaronlinv opened a new pull request #620:
URL: https://github.com/apache/dolphinscheduler-website/pull/620
[#6894](https://github.com/apache/dolphinscheduler/pull/6894)
LoggerServer has been integrated into Master and Worker, update the documentation
--
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
[GitHub] [dolphinscheduler-website] aaronlinv commented on pull request #620: Remove logger server
Posted by GitBox <gi...@apache.org>.
aaronlinv commented on pull request #620:
URL: https://github.com/apache/dolphinscheduler-website/pull/620#issuecomment-1007135444
@zhongjiajie Thanks, I have submitted the relevant changes
--
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
[GitHub] [dolphinscheduler-website] zhongjiajie commented on a change in pull request #620: Remove logger server
Posted by GitBox <gi...@apache.org>.
zhongjiajie commented on a change in pull request #620:
URL: https://github.com/apache/dolphinscheduler-website/pull/620#discussion_r780003744
##########
File path: development/en-us/development-environment-setup.md
##########
@@ -132,12 +132,11 @@ Following steps will guide how to start the DolphinScheduler backend service
##### Server start
-There are three necessary server we have to start, including MasterServer，WorkerServer，ApiApplicationServer, and a optional server you could start if you need, named LoggerServer.
+There are three services that need to be started, including MasterServer, WorkerServer, ApiApplicationServer.LoggerServer is already integrated into the MasterServer and WorkerServer, so there is no need to start them independently.
Review comment:
So as this change
##########
File path: development/zh-cn/development-environment-setup.md
##########
@@ -129,12 +129,11 @@ DolphinScheduler 的元数据存储在关系型数据库中，目前支持的关
##### 启动服务
-我们需要启动三个必须服务，包括 MasterServer，WorkerServer，ApiApplicationServer，如果有需求可以启动可选服务 LoggerServer
+我们需要启动三个服务，包括 MasterServer，WorkerServer，ApiApplicationServer，LoggerServer 已经被整合进了 MasterServer 和 WorkerServer，所以无需独立启动
Review comment:
I think we should directly remove logger server in our document, because it unrelated to general user
```suggestion
我们需要启动三个服务，包括 MasterServer，WorkerServer，ApiApplicationServer
```
--
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
[GitHub] [dolphinscheduler-website] zhongjiajie merged pull request #620: Remove logger server
Posted by GitBox <gi...@apache.org>.
zhongjiajie merged pull request #620:
URL: https://github.com/apache/dolphinscheduler-website/pull/620
--
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