Issue 3011: ignore uids on import #3045
Conversation
There was a problem hiding this comment.
Can you also resolve the conflicts?
Reviewed 16 of 16 files at r1.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @codexnull)
dgraph/cmd/live/run.go, line 102 at r1 (raw file):
flag.BoolP("use_compression", "C", false, "Enable compression on connection to alpha server") flag.Bool("new_uids", false, ""+
Is the plus sign a typo?
There was a problem hiding this comment.
Fixed.
Reviewable status: 13 of 16 files reviewed, 1 unresolved discussion (waiting on @martinmr)
dgraph/cmd/live/run.go, line 102 at r1 (raw file):
Previously, martinmr (Martin Martinez Rivera) wrote…
Is the plus sign a typo?
Yeah. Fixed.
There was a problem hiding this comment.
Reviewable status: 13 of 16 files reviewed, all discussions resolved (waiting on @martinmr)
There was a problem hiding this comment.
Reviewed 13 of 16 files at r1, 3 of 3 files at r2.
Reviewable status:complete! all files reviewed, all discussions resolved
Summary of changes:
Update bulk loader to preserve UIDs from load data by default to match live loader behavior.
Add --new_uids option to both live and bulk loaders to ignore UIDs in load data and assign new ones.
Fixes #3011.
This change is