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

release/v1.2: Support comma separated list of zero addresses in alpha (#5116) #5258

Merged
merged 1 commit into from
Apr 22, 2020

Conversation

jarifibrahim
Copy link
Contributor

@jarifibrahim jarifibrahim commented Apr 21, 2020

JIRA- DGRAPH-1132

Currently, alpha supports specifying only a single zero instance
address via the --zero flag in dgraph alpha command. This commit
adds supports for a comma-separated list of zero addresses in
dgraph alpha so that any zero address can be used from the list
of zero addresses if one of them is unavailable.

We'll pick zero addresses starting from the first one in the list and
try to establish a connection to it. If it fails, we'll pick
the next zero address.

The dgraph bulk/live command accepts only a single zero address.
The comma-separated list is only for dgraph alpha command.

(cherry picked from commit 82adb16)


This change is Reviewable

Docs Preview: Dgraph Preview

Currently, alpha supports specifying only a single zero instance
address via the `--zero` flag in `dgraph alpha` command. This commit
adds supports for a comma-separated list of `zero` addresses in
`dgraph alpha` so that any zero address can be used from the list
of zero addresses if one of them is unavailable.

We'll pick zero addresses starting from the first one in the list and
try to establish a connection to it. If it fails, we'll pick
the next zero address.

The `dgraph bulk/live` command accepts only a single zero address.
The comma-separated list is only for `dgraph alpha` command.

(cherry picked from commit 82adb16)
@jarifibrahim jarifibrahim requested review from manishrjain and a team as code owners April 21, 2020 07:19
Copy link
Contributor

@martinmr martinmr left a comment

Choose a reason for hiding this comment

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

:lgtm: also make sure that the documentation PR is also cherry-picked.

Reviewable status: 0 of 5 files reviewed, all discussions resolved (waiting on @manishrjain)

@jarifibrahim jarifibrahim merged commit 7d4ce93 into release/v1.2 Apr 22, 2020
@jarifibrahim jarifibrahim deleted the ibrahim/release-v1.2-zero-flag branch April 22, 2020 15:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants