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

Support shrink storage by specifying host #1243

Merged
merged 4 commits into from Nov 20, 2019

Conversation

critical27
Copy link
Contributor

@critical27 critical27 commented Nov 8, 2019

Support balance data between current host and the host we specified in console (we can exclude some host).

We can use in console like this:

  • Remove current machine
    BALANCE DATA REMOVE 192.168.0.1:50000,192.168.0.1:50001

@critical27 critical27 changed the title Support enlarge/shrink storage by specifying host Support shrink storage by specifying host Nov 13, 2019
@critical27 critical27 requested review from laura-ding and CPWstatic Nov 13, 2019
@critical27 critical27 added the ready-for-testing PR: ready for the CI test label Nov 13, 2019
@critical27
Copy link
Contributor Author

critical27 commented Nov 13, 2019

Jenkins go

@nebula-community-bot
Copy link
Member

nebula-community-bot commented Nov 13, 2019

Unit testing passed.

2 similar comments
@nebula-community-bot
Copy link
Member

nebula-community-bot commented Nov 13, 2019

Unit testing passed.

@nebula-community-bot
Copy link
Member

nebula-community-bot commented Nov 14, 2019

Unit testing passed.

src/meta/client/MetaClient.cpp Outdated Show resolved Hide resolved
@jude-zhu jude-zhu requested a review from wadeliuyi Nov 15, 2019
dangleptr
dangleptr previously approved these changes Nov 18, 2019
Copy link
Contributor

@dangleptr dangleptr left a comment

Well done. LGTM

@nebula-community-bot
Copy link
Member

nebula-community-bot commented Nov 18, 2019

Unit testing passed.

liuyu85cn
liuyu85cn previously approved these changes Nov 19, 2019
darionyaphet
darionyaphet previously approved these changes Nov 19, 2019
Copy link
Contributor

@darionyaphet darionyaphet left a comment

LGTM 👍

@nebula-community-bot
Copy link
Member

nebula-community-bot commented Nov 19, 2019

Unit testing failed.

@critical27
Copy link
Contributor Author

critical27 commented Nov 19, 2019

Jenkins go

@nebula-community-bot
Copy link
Member

nebula-community-bot commented Nov 19, 2019

Unit testing passed.

@nebula-community-bot
Copy link
Member

nebula-community-bot commented Nov 19, 2019

Unit testing failed.

@critical27
Copy link
Contributor Author

critical27 commented Nov 20, 2019

Jenkins go

@nebula-community-bot
Copy link
Member

nebula-community-bot commented Nov 20, 2019

Unit testing passed.

@jude-zhu jude-zhu requested review from darionyaphet and liuyu85cn Nov 20, 2019
@nebula-community-bot
Copy link
Member

nebula-community-bot commented Nov 20, 2019

Unit testing passed.

Copy link
Contributor

@dangleptr dangleptr left a comment

LGTM

@dangleptr dangleptr merged commit e407c62 into vesoft-inc:master Nov 20, 2019
@critical27 critical27 deleted the shrink branch Nov 20, 2019
@jude-zhu
Copy link
Contributor

jude-zhu commented Nov 21, 2019

close #1027

yixinglu pushed a commit to yixinglu/nebula that referenced this pull request Feb 16, 2020
tong-hao pushed a commit to tong-hao/nebula that referenced this pull request Jun 1, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-for-testing PR: ready for the CI test
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants