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.

Sign up for GitHub

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

Jump to bottom

Exports should maintain the UIDs #2999

Closed
manishrjain opened this issue Feb 10, 2019 · 1 comment
Closed

Exports should maintain the UIDs #2999

manishrjain opened this issue Feb 10, 2019 · 1 comment
Assignees
@codexnull
Labels
kind/maintenance Maintenance tasks, such as refactoring, with no impact in features. priority/P0 Critical issue that requires immediate attention.

Comments

@manishrjain
Copy link
Contributor

It causes issues for the users when Dgraph exports replace UIDs with variables, and they get reassigned to different UIDs on an export back.

My PR #2998 modifies the live loader to honor the preassigned UIDs while loading data back. So, we can now make a change to our exporter to set UIDs in the data.

Bulk loader should also do the same and honor the preassigned UIDs.
@manishrjain manishrjain added priority/P0 Critical issue that requires immediate attention. kind/maintenance Maintenance tasks, such as refactoring, with no impact in features. labels Feb 10, 2019
@codexnull codexnull self-assigned this Feb 12, 2019
@codexnull
Copy link
Contributor

Fixed with PR #3004.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@codexnull codexnull

Labels
kind/maintenance Maintenance tasks, such as refactoring, with no impact in features. priority/P0 Critical issue that requires immediate attention.
Milestone
No milestone
Development

No branches or pull requests
2 participants
@manishrjain @codexnull