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/v20.07) fix(dgraph): Fix out of order issues with split keys in bulk loader. #6124

Merged
merged 1 commit into from
Aug 3, 2020

Conversation

martinmr
Copy link
Contributor

@martinmr martinmr commented Aug 3, 2020

Split keys for indexes can cause out-of-order issues due to the variable length of the term inside the key.
To fix the issue, the split keys are written to a temporary DB first (using the writebatch to avoid the out
of order issues) and then copied to the main p directory.

Related to DGRAPH-1897

(cherry picked from commit 2a3b85c)


This change is Reviewable

Docs Preview: Dgraph Preview

…6083)

Split keys for indexes can cause out-of-order issues due to the variable length of the term inside the key.
To fix the issue, the split keys are written to a temporary DB first (using the writebatch to avoid the out
of order issues) and then copied to the main p directory.

Related to DGRAPH-1897

(cherry picked from commit 2a3b85c)
@github-actions github-actions bot added the area/bulk-loader Issues related to bulk loading. label Aug 3, 2020
@martinmr martinmr changed the title fix(dgraph): Fix out of order issues with split keys in bulk loader. (release/v20.03) fix(dgraph): Fix out of order issues with split keys in bulk loader. Aug 3, 2020
@parasssh parasssh changed the title (release/v20.03) fix(dgraph): Fix out of order issues with split keys in bulk loader. (release/v20.07) fix(dgraph): Fix out of order issues with split keys in bulk loader. Aug 3, 2020
@martinmr martinmr merged commit 3cc0646 into release/v20.07 Aug 3, 2020
@martinmr martinmr deleted the martinmr/cp-bulk07 branch August 3, 2020 17:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/bulk-loader Issues related to bulk loading.
Development

Successfully merging this pull request may close these issues.

2 participants