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

Noisy log message when error while moving tablet #4997

Closed
danielmai opened this issue Mar 21, 2020 · 1 comment
Closed

Noisy log message when error while moving tablet #4997

danielmai opened this issue Mar 21, 2020 · 1 comment
Labels
area/usability Issues with usability and error messages status/accepted We accept to investigate/work on it.

Comments

@danielmai
Copy link
Contributor

What version of Dgraph are you using?

master

Have you tried reproducing the issue with the latest release?

yes

What is the hardware spec (RAM, OS)?

Ubuntu Linux

Steps to reproduce the issue (command/config used to run Dgraph).

When move tablets are running to different groups this can show up in the Zero log:

E0321 13:45:23.570754     346 raft.go:352] While applying tablet proposal: Tablet is already being served
github.com/dgraph-io/dgraph/dgraph/cmd/zero.init
	/home/ashish/projects/src/github.com/dgraph-io/dgraph/dgraph/cmd/zero/raft.go:161
runtime.doInit
	/home/ashish/go/src/runtime/proc.go:5222
runtime.doInit
	/home/ashish/go/src/runtime/proc.go:5217
runtime.doInit
	/home/ashish/go/src/runtime/proc.go:5217
runtime.doInit
	/home/ashish/go/src/runtime/proc.go:5217
runtime.doInit
	/home/ashish/go/src/runtime/proc.go:5217
runtime.main
	/home/ashish/go/src/runtime/proc.go:190
runtime.goexit
	/home/ashish/go/src/runtime/asm_amd64.s:1357

This doesn't stop the Zero but this log with the trace makes it look like an issue.

Expected behaviour and actual result.

The error message should say that the tablet is already being served and doesn't need to show the trace.

@danielmai danielmai added status/accepted We accept to investigate/work on it. area/usability Issues with usability and error messages labels Mar 21, 2020
@lgalatin
Copy link
Contributor

lgalatin commented May 6, 2020

Fixed by #5333

@lgalatin lgalatin closed this as completed May 6, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/usability Issues with usability and error messages status/accepted We accept to investigate/work on it.
Development

No branches or pull requests

2 participants