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

optimize: add en&cn docs for ZooKeeper registry #682

Merged
merged 26 commits into from
Dec 15, 2023

Conversation

ggbocoder
Copy link
Contributor

optimize: add en&cn docs for ZooKeeper registry
fixed #399

在 /conf/registry.conf 中修改对应配置中心,其余[配置参考](https://github.com/seata/seata/blob/develop/script/client/conf/registry.conf)

```
registry {
Copy link
Contributor

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 {
Copy link
Contributor

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

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]
Copy link
Member

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.

Copy link
Contributor

@funky-eyes funky-eyes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@funky-eyes funky-eyes merged commit 43a89be into apache:docusaurus Dec 15, 2023
3 checks passed
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

Successfully merging this pull request may close these issues.

Add document using zookeeper as the registry
3 participants