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

reference/tools: update binlogctl doc to add "-show-offline-nodes" #1968

Merged
merged 8 commits into from
Oct 24, 2019

Conversation

suzaku
Copy link
Contributor

@suzaku suzaku commented Oct 22, 2019

What is changed, added or deleted?

With pingcap/tidb-binlog#774 merged, binlogct no longer shows offline pumps/drainers by default.
The related document is updated to reflected the change.

What is the related PR or file link(s)?

pingcap/tidb-binlog#774

Which version does your change affect?

  • 2.1
  • 3.0

@CLAassistant
Copy link

CLAassistant commented Oct 22, 2019

CLA assistant check
All committers have signed the CLA.

@WangXiangUSTC
Copy link
Contributor

LGTM

@suzaku suzaku requested a review from lilin90 October 22, 2019 03:52
@@ -112,6 +112,8 @@ Usage of binlogctl:
PEM 格式的 X509 key 文件的路径
-time-zone string
如果设置时区,在 "generate_meta" 模式下会打印出获取到的 tso 对应的时间。例如"Asia/Shanghai" 为 CST 时区,"Local" 为本地时区
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
如果设置时区,在 "generate_meta" 模式下会打印出获取到的 tso 对应的时间。例如"Asia/Shanghai" 为 CST 时区,"Local" 为本地时区
如果设置时区,在 "generate_meta" 模式下会打印出获取到的 tso 对应的时间。例如 "Asia/Shanghai" 为 CST 时区,"Local" 为本地时区

@lilin90 lilin90 added the translation/doing This PR’s assignee is translating this PR. label Oct 23, 2019
@lilin90 lilin90 changed the title Update binlogctl documentation to reflect that offline nodes are only shown when -show-offline-nodes is set reference/tools: update binlogctl doc to add -show-offline-nodes Oct 23, 2019
@lilin90 lilin90 changed the title reference/tools: update binlogctl doc to add -show-offline-nodes reference/tools: update binlogctl doc to add "-show-offline-nodes" Oct 23, 2019
suzaku and others added 6 commits October 23, 2019 15:53
Co-Authored-By: Lilian Lee <lilin@pingcap.com>
Co-Authored-By: Lilian Lee <lilin@pingcap.com>
Co-Authored-By: Lilian Lee <lilin@pingcap.com>
Co-Authored-By: Lilian Lee <lilin@pingcap.com>
Co-Authored-By: Lilian Lee <lilin@pingcap.com>
Copy link
Member

@lilin90 lilin90 left a comment

Choose a reason for hiding this comment

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

LGTM

@suzaku suzaku merged commit c6fc2c2 into pingcap:master Oct 24, 2019
@TomShawn TomShawn added translation/done This PR has been translated from English into Chinese and updated to pingcap/docs-cn in a PR. and removed translation/doing This PR’s assignee is translating this PR. labels Oct 30, 2019
@suzaku suzaku deleted the update-binlogctl-doc branch November 19, 2019 03:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
translation/done This PR has been translated from English into Chinese and updated to pingcap/docs-cn in a PR.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants