Skip to content
This repository was archived by the owner on Jul 24, 2024. It is now read-only.

integration_test: fix br_incremental_index test not stable (#468)#658

Merged
overvenus merged 3 commits intopingcap:release-4.0-pendingfrom
ti-srebot:release-4.0-64271cf4d16b
Dec 25, 2020
Merged

integration_test: fix br_incremental_index test not stable (#468)#658
overvenus merged 3 commits intopingcap:release-4.0-pendingfrom
ti-srebot:release-4.0-64271cf4d16b

Conversation

@ti-srebot
Copy link
Contributor

cherry-pick #468 to release-4.0


What problem does this PR solve?

resolve #454

What is changed and how it works?

we shouldn't filter schemas that doesn't have backup files. because the schema may have ddl jobs to do.

Check List

Tests

  • Integration test

Code changes

  • Has exported function/method change
  • Has exported variable/fields change
  • Has interface methods change
  • Has persistent data change

Side effects

  • Possible performance regression
  • Increased code complexity
  • Breaking backward compatibility

Related changes

  • Need to cherry-pick to the release branch

Release Note

  • No release note

Signed-off-by: ti-srebot <ti-srebot@pingcap.com>
@kennytm
Copy link
Collaborator

kennytm commented Dec 25, 2020

LGTM

@ti-srebot ti-srebot added the status/LGT1 LGTM1 label Dec 25, 2020
@overvenus overvenus changed the base branch from release-4.0 to release-4.0-pending December 25, 2020 08:36
@overvenus
Copy link
Member

LGTM

@ti-srebot ti-srebot added status/LGT2 LGTM2 and removed status/LGT1 LGTM1 labels Dec 25, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants