Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@0xPoe
Copy link
Member

@0xPoe 0xPoe commented Oct 21, 2021

What problem does this PR solve?

Prepare for more old value tests.

What is changed and how it works?

rename new ci collation test to new ci collation with old value test.

Check List

Tests

  • Integration test

Code changes

None

Side effects

None

Related changes

  • Need to cherry-pick to the release branch

Release note

None

@ti-chi-bot
Copy link
Member

ti-chi-bot commented Oct 21, 2021

[REVIEW NOTIFICATION]

This pull request has been approved by:

  • ben1009
  • overvenus

To complete the pull request process, please ask the reviewers in the list to review by filling /cc @reviewer in the comment.
After your PR has acquired the required number of LGTMs, you can assign this pull request to the committer in the list by filling /assign @committer in the comment to help you merge this pull request.

The full list of commands accepted by this bot can be found here.

Reviewer can indicate their review by submitting an approval review.
Reviewer can cancel approval by submitting a request changes review.

@ti-chi-bot ti-chi-bot added release-note-none Denotes a PR that doesn't merit a release note. size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Oct 21, 2021
@0xPoe
Copy link
Member Author

0xPoe commented Oct 21, 2021

/run-all-tests

@0xPoe 0xPoe requested review from ben1009 and overvenus October 21, 2021 08:11
@0xPoe
Copy link
Member Author

0xPoe commented Oct 21, 2021

/run-unit-tests

@ti-chi-bot ti-chi-bot added size/L Denotes a PR that changes 100-499 lines, ignoring generated files. and removed size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Oct 21, 2021
@0xPoe 0xPoe changed the title test: rename new ci collation test test: add new ci collation without old value test Oct 21, 2021
@0xPoe
Copy link
Member Author

0xPoe commented Oct 21, 2021

/run-all-tests

@ti-chi-bot ti-chi-bot added the status/LGT1 Indicates that a PR has LGTM 1. label Oct 21, 2021
@0xPoe
Copy link
Member Author

0xPoe commented Oct 21, 2021

/run-all-tests

@0xPoe 0xPoe changed the title test: add new ci collation without old value test WIP: test: add new ci collation without old value test Oct 21, 2021
@ti-chi-bot ti-chi-bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Oct 21, 2021
@amyangfei amyangfei added this to the v5.3.0 milestone Oct 21, 2021
@0xPoe
Copy link
Member Author

0xPoe commented Oct 21, 2021

/run-all-tests

@0xPoe
Copy link
Member Author

0xPoe commented Oct 21, 2021

/run-kafka-tests

@0xPoe
Copy link
Member Author

0xPoe commented Oct 21, 2021

/run-all-tests

@0xPoe
Copy link
Member Author

0xPoe commented Oct 21, 2021

/run-all-tests

@overvenus
Copy link
Member

/run-kafka-tests
/run-integration-tests

1 similar comment
@0xPoe
Copy link
Member Author

0xPoe commented Oct 21, 2021

/run-kafka-tests
/run-integration-tests

@0xPoe
Copy link
Member Author

0xPoe commented Oct 21, 2021

/run-kafka-tests

4 similar comments
@0xPoe
Copy link
Member Author

0xPoe commented Oct 21, 2021

/run-kafka-tests

@amyangfei
Copy link
Contributor

/run-kafka-tests

@0xPoe
Copy link
Member Author

0xPoe commented Oct 21, 2021

/run-kafka-tests

@0xPoe
Copy link
Member Author

0xPoe commented Oct 21, 2021

/run-kafka-tests

@ti-chi-bot ti-chi-bot merged commit 7bba941 into pingcap:master Oct 21, 2021
@0xPoe 0xPoe deleted the rustin-patch-test-new-collation branch October 21, 2021 15:06
@0xPoe 0xPoe added needs-cherry-pick-release-4.0 Should cherry pick this PR to release-4.0 branch. needs-cherry-pick-release-5.0 Should cherry pick this PR to release-5.0 branch. needs-cherry-pick-release-5.1 Should cherry pick this PR to release-5.1 branch. needs-cherry-pick-release-5.2 Should cherry pick this PR to release-5.2 branch. labels Nov 10, 2021
ti-chi-bot pushed a commit to ti-chi-bot/tiflow that referenced this pull request Nov 10, 2021
@ti-chi-bot
Copy link
Member

In response to a cherrypick label: new pull request created: #3368.

@ti-chi-bot
Copy link
Member

In response to a cherrypick label: new pull request created: #3367.

@ti-chi-bot
Copy link
Member

In response to a cherrypick label: new pull request created: #3369.

@ti-chi-bot
Copy link
Member

In response to a cherrypick label: new pull request created: #3370.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs-cherry-pick-release-4.0 Should cherry pick this PR to release-4.0 branch. needs-cherry-pick-release-5.0 Should cherry pick this PR to release-5.0 branch. needs-cherry-pick-release-5.1 Should cherry pick this PR to release-5.1 branch. needs-cherry-pick-release-5.2 Should cherry pick this PR to release-5.2 branch. release-note-none Denotes a PR that doesn't merit a release note. size/L Denotes a PR that changes 100-499 lines, ignoring generated files. status/can-merge Indicates a PR has been approved by a committer. status/LGT2 Indicates that a PR has LGTM 2.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants