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

skip wrongly write ActionUpdateTiFlashReplicaStatus job #947

Merged
merged 2 commits into from
Apr 7, 2020
Merged

Conversation

july2993
Copy link
Contributor

@july2993 july2993 commented Apr 7, 2020

What problem does this PR solve?

ref #941

What is changed and how it works?

skip wrongly write ActionUpdateTiFlashReplicaStatus job

after pingcap/tidb#16049 there's no any ActionUpdateTiFlashReplicaStatus job that will be written.

Check List

Tests

  • Integration test
  • Manual test (add detailed scripts or steps below)
    create table t(id int);
    ALTER TABLE t SET TIFLASH REPLICA 3 LOCATION LABELS "rack", "host", "abc"; // model.ActionSetTiFlashReplica
    curl -X POST -d '{"id":45,"region_count":3,"flash_region_count":3}' https://127.0.0.1:10080/tiflash/replica // model.ActionUpdateTiFlashReplicaStatus
    performance write on table t and check can still replicate. - No code

Code changes

Side effects

Related changes

@july2993
Copy link
Contributor Author

july2993 commented Apr 7, 2020

/run-all-tests tidb=release-4.0 tikv=release-4.0 pd=release-4.0

@july2993
Copy link
Contributor Author

july2993 commented Apr 7, 2020

/run-all-tests tidb=release-4.0 tikv=release-4.0 pd=release-4.0

@july2993 july2993 changed the title Fix tiflash skip wrongly write ActionUpdateTiFlashReplicaStatus job Apr 7, 2020
@july2993
Copy link
Contributor Author

july2993 commented Apr 7, 2020

/run-all-tests tidb=release-4.0 tikv=release-4.0 pd=release-4.0

@july2993
Copy link
Contributor Author

july2993 commented Apr 7, 2020

/run-unit-test

@july2993
Copy link
Contributor Author

july2993 commented Apr 7, 2020

/run-all-tests tidb=release-4.0 tikv=release-4.0 pd=release-4.0

Copy link
Collaborator

@IANTHEREAL IANTHEREAL left a comment

Choose a reason for hiding this comment

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

LGTM

@sre-bot
Copy link

sre-bot commented Apr 7, 2020

cherry pick to release-3.1 in PR #948

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.

3 participants