-
Notifications
You must be signed in to change notification settings - Fork 114
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
optimize: add en&cn docs for ZooKeeper registry #682
Conversation
在 /conf/registry.conf 中修改对应配置中心,其余[配置参考](https://github.com/seata/seata/blob/develop/script/client/conf/registry.conf) | ||
|
||
``` | ||
registry { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
要用最新版写,1.5开始全部用的是application.yml
You have to write it in the latest version, since 1.5 it's all application.yml.
在 /conf/registry.conf 中修改对应配置中心,其余[配置参考](https://github.com/seata/seata/blob/develop/script/client/conf/registry.conf) | ||
|
||
``` | ||
registry { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
这种方式在1.5以下都是通用的,所以上面描述中不需要特别指出是1.4.2版本下进行撰写的
This approach is common up to 1.5, so there is no need to specifically point out in the description above that it was written under version 1.4.2
i18n/en/docusaurus-plugin-content-docs/version-v1.0/user/registry/zookeeper.md
Outdated
Show resolved
Hide resolved
# Conflicts: # yarn.lock
Client 配置完成后启动应用并稍待片刻,出现以下后日志就可以正式体验 Seata 服务 | ||
|
||
```text | ||
register TM success. client version:1.4.2, server version:1.4.2,channel:[id: 0xa4675e28, L:/127.0.0.1:8238 - R:/127.0.0.1:8091] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This log is too heavily coupled to the version, so it is recommended to optimize it. If it is necessary, please provide key logs that are independent of the version.
i18n/zh-cn/docusaurus-plugin-content-docs/current/user/registry/zookeeper.md
Show resolved
Hide resolved
i18n/zh-cn/docusaurus-plugin-content-docs/current/user/registry/zookeeper.md
Outdated
Show resolved
Hide resolved
i18n/zh-cn/docusaurus-plugin-content-docs/current/user/registry/zookeeper.md
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
optimize: add en&cn docs for ZooKeeper registry
fixed #399