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.

Sign up for GitHub

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

Jump to bottom

PD will panic if tikv enable forwarding #5373

Closed
bufferflies opened this issue Jul 28, 2022 · 0 comments · Fixed by #5374
Closed

PD will panic if tikv enable forwarding #5373

bufferflies opened this issue Jul 28, 2022 · 0 comments · Fixed by #5374
Labels
affects-6.1 affects-6.2 severity/major type/bug The issue is confirmed as a bug.

Comments

@bufferflies
Copy link
Contributor

Bug Report

image

What did you do?

enable forwarding for tikv

What did you expect to see?

nomal

What did you see instead?

panic

What version of PD are you using (pd-server -V)?

master
@bufferflies bufferflies added the type/bug The issue is confirmed as a bug. label Jul 28, 2022
@bufferflies bufferflies mentioned this issue Jul 28, 2022
Merged
ti-chi-bot pushed a commit that referenced this issue Jul 28, 2022
@bufferflies @nolouch
grpc: fix the wrong error handler. (#5374)
e949ddf 
close #5373

Signed-off-by: bufferflies <1045931706@qq.com>
Signed-off-by: nolouch <nolouch@gmail.com>

Co-authored-by: nolouch <nolouch@gmail.com>
ti-chi-bot pushed a commit to ti-chi-bot/pd that referenced this issue Jul 28, 2022
@bufferflies @ti-chi-bot
This is an automated cherry-pick of tikv#5374
6de0fb6 
close tikv#5373

Signed-off-by: ti-chi-bot <ti-community-prow-bot@tidb.io>
This was referenced Jul 28, 2022
Merged
Merged
nolouch pushed a commit to nolouch/pd that referenced this issue Jul 29, 2022
@bufferflies @nolouch
grpc: fix the wrong error handler. (tikv#5374)
fc66b76 
close tikv#5373

Signed-off-by: bufferflies <1045931706@qq.com>
Signed-off-by: nolouch <nolouch@gmail.com>

Co-authored-by: nolouch <nolouch@gmail.com>
Conflicts:
	tests/client/client_test.go
@nolouch nolouch mentioned this issue Jul 29, 2022
Merged
nolouch added a commit that referenced this issue Jul 29, 2022
@nolouch @bufferflies
grpc: fix the wrong error handler. (#5374) (#5378)
355242c 
* grpc eror handle

Signed-off-by: bufferflies <1045931706@qq.com>

* grpc: fix the wrong error handler. (#5374)

close #5373

Signed-off-by: bufferflies <1045931706@qq.com>
Signed-off-by: nolouch <nolouch@gmail.com>

Co-authored-by: nolouch <nolouch@gmail.com>
Conflicts:
	tests/client/client_test.go

Co-authored-by: bufferflies <1045931706@qq.com>
ti-chi-bot added a commit that referenced this issue Jul 29, 2022
@ti-chi-bot @bufferflies @nolouch
grpc: fix the wrong error handler. (#5374) (#5377)
989134e 
close #5373, ref #5374

Signed-off-by: bufferflies <1045931706@qq.com>
Signed-off-by: nolouch <nolouch@gmail.com>

Co-authored-by: bufferflies <1045931706@qq.com>
Co-authored-by: nolouch <nolouch@gmail.com>
ti-chi-bot added a commit that referenced this issue Aug 1, 2022
@ti-chi-bot @bufferflies
grpc: fix the wrong error handler. (#5374) (#5376)
c91d5ca 
close #5373, ref #5374

Signed-off-by: ti-chi-bot <ti-community-prow-bot@tidb.io>
Signed-off-by: bufferflies <1045931706@qq.com>

Co-authored-by: buffer <1045931706@qq.com>
Co-authored-by: bufferflies <1045931706@qq.com>
@nolouch nolouch mentioned this issue Aug 29, 2022
Merged
12 tasks
@nolouch nolouch mentioned this issue Sep 26, 2022
Merged
13 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
affects-6.1 affects-6.2 severity/major type/bug The issue is confirmed as a bug.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

grpc: fix the wrong error handler. bufferflies/pd
3 participants
@nolouch @bufferflies @ti-chi-bot