Skip to content

propose key same cause assert panic[Found existing proposal with key: [7d02e530d09]] #7484

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

Closed
wants to merge 1 commit into from

Conversation

yangxuanjia
Copy link

@yangxuanjia yangxuanjia commented Feb 25, 2021

2021/02/02 13:09:51 Found existing proposal with key: [7d02e530d09]
github.com/dgraph-io/dgraph/x.AssertTruef
        /export/working/src/github.com/dgraph-io/dgraph/x/error.go:107
github.com/dgraph-io/dgraph/worker.(*node).proposeAndWait.func3
        /export/working/src/github.com/dgraph-io/dgraph/worker/proposal.go:221
github.com/dgraph-io/dgraph/worker.(*node).proposeAndWait.func4
        /export/working/src/github.com/dgraph-io/dgraph/worker/proposal.go:289
github.com/dgraph-io/dgraph/worker.(*node).proposeAndWait
        /export/working/src/github.com/dgraph-io/dgraph/worker/proposal.go:293
github.com/dgraph-io/dgraph/worker.(*grpcWorker).proposeAndWait
        /export/working/src/github.com/dgraph-io/dgraph/worker/mutation.go:814
github.com/dgraph-io/dgraph/worker.proposeOrSend
        /export/working/src/github.com/dgraph-io/dgraph/worker/mutation.go:538
runtime.goexit
        /export/go/src/runtime/asm_amd64.s:1373
[Decoder]: Using assembly version of decoder
Page Size: 4096

This change is Reviewable

Copy link
Contributor

@NamanJain8 NamanJain8 left a comment

Choose a reason for hiding this comment

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

Thanks, @yangxuanjia , looks good to me. Dgraph should not crash in random number collision.

@NamanJain8
Copy link
Contributor

This is not needed after this change #7741 as we are generating better non-random numbers (slightly randomized by initUniqueKey). Thanks @yangxuanjia.

@NamanJain8 NamanJain8 closed this May 13, 2021
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