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

*: Update installation.md & add an exit code in start_zk #324

Merged
merged 4 commits into from
May 16, 2019

Conversation

vagetablechicken
Copy link
Contributor

No description provided.

@vagetablechicken vagetablechicken marked this pull request as ready for review May 8, 2019 06:46
qinzuoyan
qinzuoyan previously approved these changes May 9, 2019
docs/installation.md Outdated Show resolved Hide resolved
@vagetablechicken
Copy link
Contributor Author

build dependencies在ubuntu18.04和centos7上均做了验证。附带修复了一下start_zk的逻辑,start_zk失败后不会start servers。

@vagetablechicken vagetablechicken requested review from acelyc111 and qinzuoyan and removed request for acelyc111 May 10, 2019 10:09
@vagetablechicken vagetablechicken changed the title Update installation.md *: Update installation.md & add an exit code in start_zk May 10, 2019
@vagetablechicken vagetablechicken merged commit 3659ea7 into master May 16, 2019
@vagetablechicken vagetablechicken deleted the vagetablechicken-patch-1 branch May 16, 2019 09:50
@neverchanje neverchanje mentioned this pull request Jun 11, 2019
24 tasks
neverchanje pushed a commit that referenced this pull request Jun 14, 2019
* dependencies fix

* exit 1 if start_zk failed
@neverchanje neverchanje mentioned this pull request Jun 21, 2019
19 tasks
neverchanje pushed a commit that referenced this pull request Jun 21, 2019
* dependencies fix

* exit 1 if start_zk failed
neverchanje pushed a commit that referenced this pull request Jul 1, 2019
* dependencies fix

* exit 1 if start_zk failed
neverchanje pushed a commit to neverchanje/pegasus that referenced this pull request Jul 13, 2019
* dependencies fix

* exit 1 if start_zk failed


Former-commit-id: d6e150dc9f0efe57c385639ee8e349ff1db86e82 [formerly 3659ea7]
Former-commit-id: 448daeaa672884e9ea0cab2bb7e5b967d4035843
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants