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

add index failed with error “request pd http api failed with status: '500 Internal Server Error', body: '[PD:apiutil:ErrRedirect]redirect failed” when kill pd leader #8499

Closed
Lily2025 opened this issue Aug 6, 2024 · 7 comments
Closed

add index failed with error “request pd http api failed with status: '500 Internal Server Error', body: '[PD:apiutil:ErrRedirect]redirect failed” when kill pd leader #8499

Lily2025 opened this issue Aug 6, 2024 · 7 comments
Assignees
@JmPotato
Labels
affects-8.1 This bug affects the 8.1.x(LTS) versions. severity/major type/bug The issue is confirmed as a bug.

Comments

@Lily2025
Copy link

Lily2025 commented Aug 6, 2024

Bug Report

What did you do?

1、tidb_enable_dist_task='on'
2、run sysbench
3、add index for one table
4、kill pd leader

What did you expect to see?

add index can success

What did you see instead?

add index failed with error “request pd http api failed with status: '500 Internal Server Error', body: '[PD:apiutil:ErrRedirect]redirect failed”

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

./pd-server -V
Release Version: v8.1.1
Edition: Community
Git Commit Hash: 3d11c4d
Git Branch: HEAD
UTC Build Time: 2024-08-05 12:23:13
2024-08-06T06:24:22.864+0800
@Lily2025 Lily2025 added the type/bug The issue is confirmed as a bug. label Aug 6, 2024
@Lily2025
Copy link
Author

Lily2025 commented Aug 6, 2024

/severity major

@Lily2025
Copy link
Author

Lily2025 commented Aug 6, 2024

/assign JmPotato

@Lily2025
Copy link
Author

Lily2025 commented Aug 6, 2024

#8465

@Lily2025
Copy link
Author

Lily2025 commented Aug 6, 2024

/remove-label may-affects-5.4
/remove-label may-affects-6.1
/remove-label may-affects-6.5
/remove-label may-affects-7.1
/remove-label may-affects-7.5

@Lily2025
Copy link
Author

Lily2025 commented Aug 6, 2024

/label affects-8.1

@ti-chi-bot ti-chi-bot bot added affects-8.1 This bug affects the 8.1.x(LTS) versions. and removed may-affects-8.1 labels Aug 6, 2024
@Lily2025
Copy link
Author

Lily2025 commented Aug 6, 2024

/remove-label may-affects-8.1

@ti-chi-bot ti-chi-bot
Copy link
Contributor

ti-chi-bot bot commented Aug 6, 2024

@Lily2025: These labels are not set on the issue: may-affects-8.1.

In response to this:

/remove-label may-affects-8.1

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the ti-community-infra/tichi repository.

@JmPotato JmPotato mentioned this issue Aug 6, 2024
Merged
ti-chi-bot bot pushed a commit that referenced this issue Aug 6, 2024
@ti-chi-bot @JmPotato
client/retry: only return the latest error in backoffer (#8227) (#8465)
10ecdbe 
ref #8142, close #8499

Due to the return of historical errors causing the client's retry logic to fail,
and since we currently do not need to obtain all errors during retries, this PR
removes `multierr` from backoffer and add tests to ensure the correctness of the retry logic.

Signed-off-by: JmPotato <ghzpotato@gmail.com>

Co-authored-by: JmPotato <ghzpotato@gmail.com>
@JmPotato JmPotato closed this as completed Aug 7, 2024
@niubell niubell mentioned this issue Dec 16, 2024
Merged
16 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@JmPotato JmPotato

Labels
affects-8.1 This bug affects the 8.1.x(LTS) versions. severity/major type/bug The issue is confirmed as a bug.
Projects
Questions and Bug Reports
Status: Closed
Milestone
No milestone
Development

No branches or pull requests
2 participants
@JmPotato @Lily2025