Skip to content

fix(mutation): Mutation request stuck when --limit flag is set #8656

Merged
mangalaman93 merged 1 commit intomainfrom
shiva/limit
Feb 8, 2023
Merged

fix(mutation): Mutation request stuck when --limit flag is set #8656
mangalaman93 merged 1 commit intomainfrom
shiva/limit

Conversation

@shivaji-kharse
Copy link
Copy Markdown
Contributor

@shivaji-kharse shivaji-kharse commented Feb 8, 2023

@coveralls
Copy link
Copy Markdown

coveralls commented Feb 8, 2023

Coverage Status

Coverage: 66.659% (-0.4%) from 67.071% when pulling c636e6e on shiva/limit into a153ed5 on main.

Comment thread dgraph/cmd/zero/assign.go
//nolint:gosec // random generator in closed set does not require cryptographic precision
delay := rand.Intn(int(opts.limiterConfig.RefillAfter))
time.Sleep(time.Duration(delay) * time.Second)
time.Sleep(time.Duration(delay))
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Wow, good catch!

@mangalaman93 mangalaman93 merged commit 3c100d5 into main Feb 8, 2023
@mangalaman93 mangalaman93 deleted the shiva/limit branch February 8, 2023 20:20
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