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 live loader flag help text. #3278

Merged
merged 2 commits into from
Apr 10, 2019
Merged

Update live loader flag help text. #3278

merged 2 commits into from
Apr 10, 2019

Conversation

danielmai
Copy link
Contributor

@danielmai danielmai commented Apr 10, 2019

The live loader can already take in multiple Dgraph Alpha addresses to
distribute the load across the cluster during loading. The flag help text makes
this option clearer.


This change is Reviewable

The live loader can already take in multiple Dgraph Alpha addresses to
distribute the load across the cluster during loading. The flag help text makes
this option clear.
@danielmai danielmai requested a review from a team April 10, 2019 17:00
@@ -87,7 +87,7 @@ func init() {
flag.StringP("files", "f", "", "Location of *.rdf(.gz) or *.json(.gz) file(s) to load")
flag.StringP("schema", "s", "", "Location of schema file")
flag.String("format", "", "Specify file format (rdf or json) instead of getting it from filename")
flag.StringP("dgraph", "d", "127.0.0.1:9080", "Dgraph alpha gRPC server address")
flag.StringP("dgraph", "d", "127.0.0.1:9080", "Comma-separated list of Dgraph alpha gRPC server addresses")

Choose a reason for hiding this comment

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

line is 108 characters (from lll)

Copy link
Contributor

@codexnull codexnull left a comment

Choose a reason for hiding this comment

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

Reviewed 1 of 1 files at r2.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @danielmai)

Copy link
Contributor

@manishrjain manishrjain left a comment

Choose a reason for hiding this comment

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

:lgtm:

Reviewed 1 of 1 files at r2.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @danielmai)

@danielmai danielmai merged commit da84554 into master Apr 10, 2019
@danielmai danielmai deleted the danielmai/live-flag branch April 10, 2019 19:43
MichelDiz added a commit that referenced this pull request Apr 12, 2019
Added about distributed load related to #3278
Change in increment section is related to #3284
MichelDiz added a commit that referenced this pull request Apr 12, 2019
* Docs: Distributed Load and change increment flag

Added about distributed load related to #3278
Change in increment section is related to #3284

* docs: Update description for live loader's alpha flag.
dna2github pushed a commit to dna2fork/dgraph that referenced this pull request Jul 19, 2019
The live loader can already take in multiple Dgraph Alpha addresses to
distribute the load across the cluster during loading. The flag help text makes
this option clear.
dna2github pushed a commit to dna2fork/dgraph that referenced this pull request Jul 19, 2019
* Docs: Distributed Load and change increment flag

Added about distributed load related to dgraph-io#3278
Change in increment section is related to dgraph-io#3284

* docs: Update description for live loader's alpha flag.
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.

4 participants