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-cloud: opt(dropPrefix): allow logical drop for deleting predicates and indexing (#7764) #7779

Merged
merged 2 commits into from
May 5, 2021

Conversation

NamanJain8
Copy link
Contributor

@NamanJain8 NamanJain8 commented May 5, 2021

Issue:
DropPrefix is Stop The World operation. It blocks the incoming writes. Dgraph uses this API at a couple of places like:

  • drop data
  • reindexing
  • dropping predicate
    The last 2 usages can lead to bad user impact in a multi-tenant cluster.

Solution:
This PR allows DropPrefix to use the blocking/non-blocking DropPrefix based on DB option AllowStopTheWorld that can be set using --badger superflag.

(cherry picked from commit c86b855)


This change is Reviewable

…ing (#7764)

Issue:
DropPrefix is Stop The World operation. It blocks the incoming writes. Dgraph uses this API at a couple of places like:
- drop data
- reindexing
- dropping predicate
The last 2 usages can lead to bad user impact in a multi-tenant cluster.

Solution:
This PR allows DropPrefix to use the blocking/non-blocking DropPrefix based on DB option `AllowStopTheWorld` that can be set using `--badger` superflag.

(cherry picked from commit c86b855)
@NamanJain8 NamanJain8 merged commit 03204ee into release/v21.03-slash May 5, 2021
@NamanJain8 NamanJain8 deleted the naman/cp/drop-prefix branch May 5, 2021 15:26
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