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

server: GetAllStores should not return tombstone stores (#1440) #1444

Merged
merged 1 commit into from
Feb 27, 2019

Conversation

nolouch
Copy link
Contributor

@nolouch nolouch commented Feb 26, 2019

What problem does this PR solve?

cherry pick #1440

@nolouch
Copy link
Contributor Author

nolouch commented Feb 26, 2019

/run-all-tests tidb=release-2.1 tikv=release-2.1 tidb-test=release-2.1

@nolouch nolouch added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Feb 26, 2019
@overvenus
Copy link
Member

pingcap/kvproto#361 has been merged, could you update kvproto?

@nolouch nolouch force-pushed the pick-1440 branch 2 times, most recently from 4b84d0d to 2b8af24 Compare February 26, 2019 07:29
* server: add option to exclude tombstone stores

GetAllStores should not return tombstone stores in most cases. We can
add an option to support that if we needs it later.

Signed-off-by: Huachao Huang <huachao.huang@gmail.com>
@nolouch nolouch removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Feb 26, 2019
@nolouch
Copy link
Contributor Author

nolouch commented Feb 26, 2019

/run-all-tests tidb=release-2.1 tikv=release-2.1 tidb-test=release-2.1

1 similar comment
@nolouch
Copy link
Contributor Author

nolouch commented Feb 26, 2019

/run-all-tests tidb=release-2.1 tikv=release-2.1 tidb-test=release-2.1

@zhouqiang-cl
Copy link
Contributor

/rebuild

1 similar comment
@zhouqiang-cl
Copy link
Contributor

/rebuild

@zhouqiang-cl
Copy link
Contributor

/run-all-tests tidb=release-2.1 tikv=release-2.1 tidb-test=release-2.1

@zhouqiang-cl
Copy link
Contributor

/rebuild

@zhouqiang-cl
Copy link
Contributor

/run-all-tests tidb=release-2.1 tikv=release-2.1 tidb-test=release-2.1

1 similar comment
@zhouqiang-cl
Copy link
Contributor

/run-all-tests tidb=release-2.1 tikv=release-2.1 tidb-test=release-2.1

Copy link
Contributor

@huachaohuang huachaohuang left a comment

Choose a reason for hiding this comment

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

LGTM

@disksing disksing merged commit 5eac798 into tikv:release-2.1 Feb 27, 2019
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.

6 participants