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

Translate tidb computing #3057

Merged
merged 63 commits into from
Jul 20, 2020
Merged

Translate tidb computing #3057

merged 63 commits into from
Jul 20, 2020

Conversation

baurine
Copy link
Contributor

@baurine baurine commented Jun 28, 2020

What is changed, added or deleted? (Required)

Translate the TiDB Computation chapter from Chinese to English.
This PR should be merged after #3094 for the TOC to be updated.

Which TiDB version(s) do your changes apply to? (Required)

  • master (the latest development version)
  • v4.0 (TiDB 4.0 versions)
  • v3.1 (TiDB 3.1 versions)
  • v3.0 (TiDB 3.0 versions)
  • v2.1 (TiDB 2.1 versions)

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

@CLAassistant
Copy link

CLAassistant commented Jun 28, 2020

CLA assistant check
All committers have signed the CLA.

@github-actions github-actions bot requested a review from TomShawn June 28, 2020 13:45
@github-actions github-actions bot added translation/welcome Waits for a contributor to translate this PR and create a PR to the pingcap/docs-cn repository. status/PTAL This PR is ready for reviewing. labels Jun 28, 2020
@baurine
Copy link
Contributor Author

baurine commented Jun 28, 2020

@SunRunAway @yikeke PTAL, thanks!

@TomShawn TomShawn requested a review from SunRunAway June 28, 2020 13:52
@TomShawn TomShawn added needs-cherry-pick-4.0 translation/from-docs-cn This PR is translated from a PR in pingcap/docs-cn. and removed translation/welcome Waits for a contributor to translate this PR and create a PR to the pingcap/docs-cn repository. labels Jun 28, 2020
@ti-srebot
Copy link
Contributor

@SunRunAway, @TomShawn, PTAL.

2 similar comments
@ti-srebot
Copy link
Contributor

@SunRunAway, @TomShawn, PTAL.

@ti-srebot
Copy link
Contributor

@SunRunAway, @TomShawn, PTAL.

@TomShawn TomShawn added the special-week PR from Document Special Week. label Jul 6, 2020
Copy link
Contributor

@TomShawn TomShawn left a comment

Choose a reason for hiding this comment

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

Good translation! I'll review the rest part soon.

tidb-computing.md Outdated Show resolved Hide resolved
tidb-computing.md Outdated Show resolved Hide resolved
tidb-computing.md Outdated Show resolved Hide resolved
tidb-computing.md Outdated Show resolved Hide resolved
tidb-computing.md Outdated Show resolved Hide resolved
tidb-computing.md Outdated Show resolved Hide resolved
tidb-computing.md Outdated Show resolved Hide resolved
tidb-computing.md Outdated Show resolved Hide resolved
tidb-computing.md Outdated Show resolved Hide resolved
tidb-computing.md Outdated Show resolved Hide resolved
@TomShawn TomShawn added status/require-change Needs the author to address comments. and removed status/PTAL This PR is ready for reviewing. labels Jul 6, 2020
tidb-computing.md Outdated Show resolved Hide resolved
tidb-computing.md Outdated Show resolved Hide resolved
Copy link
Contributor

@TomShawn TomShawn left a comment

Choose a reason for hiding this comment

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

The second round of review

tidb-computing.md Outdated Show resolved Hide resolved
tidb-computing.md Outdated Show resolved Hide resolved
tidb-computing.md Outdated Show resolved Hide resolved
tidb-computing.md Outdated Show resolved Hide resolved
tidb-computing.md Outdated Show resolved Hide resolved
tidb-computing.md Outdated Show resolved Hide resolved
tidb-computing.md Outdated Show resolved Hide resolved
tidb-computing.md Outdated Show resolved Hide resolved
tidb-computing.md Outdated Show resolved Hide resolved
tidb-computing.md Outdated Show resolved Hide resolved
@ti-srebot
Copy link
Contributor

@baurine, please update your pull request.

@TomShawn
Copy link
Contributor

LGTM
@SunRunAway PTAL

@ti-srebot
Copy link
Contributor

@TomShawn, @SunRunAway, PTAL.

@ti-srebot
Copy link
Contributor

@baurine, please update your pull request.

@baurine
Copy link
Contributor Author

baurine commented Jul 15, 2020

Err, seems I have no permission to push code, what should I continue to do for this PR?

@ti-srebot
Copy link
Contributor

@TomShawn, @SunRunAway, PTAL.

TOC.md Outdated Show resolved Hide resolved
@ti-srebot
Copy link
Contributor

@TomShawn, @SunRunAway, PTAL.

@ti-srebot
Copy link
Contributor

@baurine, please update your pull request.

tidb-computing.md Outdated Show resolved Hide resolved
Co-authored-by: Feng Liyuan <darktemplar.f@gmail.com>
Copy link
Contributor

@SunRunAway SunRunAway left a comment

Choose a reason for hiding this comment

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

LGTM

@ti-srebot
Copy link
Contributor

@SunRunAway,Thanks for your review. However, LGTM is restricted to Reviewers or higher roles.See the corresponding SIG page for more information. Related SIGs: docs(slack).

Copy link
Contributor

@TomShawn TomShawn left a comment

Choose a reason for hiding this comment

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

LGTM

@ti-srebot ti-srebot added the status/LGT1 Indicates that a PR has LGTM 1. label Jul 20, 2020
@TomShawn TomShawn merged commit 38620d3 into master Jul 20, 2020
ti-srebot pushed a commit to ti-srebot/docs that referenced this pull request Jul 20, 2020
Signed-off-by: ti-srebot <ti-srebot@pingcap.com>
@ti-srebot ti-srebot mentioned this pull request Jul 20, 2020
5 tasks
@ti-srebot
Copy link
Contributor

cherry pick to release-4.0 in PR #3341

TomShawn added a commit that referenced this pull request Jul 20, 2020
* cherry pick #3057 to release-4.0

Signed-off-by: ti-srebot <ti-srebot@pingcap.com>

* Apply suggestions from code review

Co-authored-by: Sparkle <1284531+baurine@users.noreply.github.com>
Co-authored-by: TomShawn <41534398+TomShawn@users.noreply.github.com>
@baurine baurine deleted the translate-tidb-computing branch July 20, 2020 12:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
special-week PR from Document Special Week. status/LGT1 Indicates that a PR has LGTM 1. status/require-change Needs the author to address comments. translation/from-docs-cn This PR is translated from a PR in pingcap/docs-cn.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants