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

force_init_stats doesn't block HTTP API's startup #50854

Closed
hawkingrei opened this issue Jan 31, 2024 · 0 comments · Fixed by #50853
Closed

force_init_stats doesn't block HTTP API's startup #50854

hawkingrei opened this issue Jan 31, 2024 · 0 comments · Fixed by #50853
Labels
affects-6.5 This bug affects the 6.5.x(LTS) versions. affects-7.1 This bug affects the 7.1.x(LTS) versions. affects-7.5 This bug affects the 7.5.x(LTS) versions. sig/planner SIG: Planner type/enhancement The issue or PR belongs to an enhancement.

Comments

@hawkingrei
Copy link
Member

Enhancement

In the current implementation, the force_init_stats block the startup of HTTP APIs. This could potentially lead to issues as the API cannot be accessed before the stats are properly initialized.
@hawkingrei hawkingrei added the type/enhancement The issue or PR belongs to an enhancement. label Jan 31, 2024
@hawkingrei hawkingrei mentioned this issue Jan 31, 2024
Merged
13 tasks
ti-chi-bot bot pushed a commit that referenced this issue Feb 2, 2024
@hawkingrei
*: force_init_stats doesn't block http api's startup (#50853)
b7ce239 
close #50854
This was referenced Feb 7, 2024
Merged
Merged
Merged
@ti-chi-bot ti-chi-bot added affects-6.5 This bug affects the 6.5.x(LTS) versions. affects-7.5 This bug affects the 7.5.x(LTS) versions. affects-7.1 This bug affects the 7.1.x(LTS) versions. labels Feb 7, 2024
ti-chi-bot bot pushed a commit that referenced this issue Feb 18, 2024
@ti-chi-bot
*: force_init_stats doesn't block http api's startup (#50853) (#51040)
e1f9450 
close #50854
@jebter jebter added the sig/planner SIG: Planner label Feb 22, 2024
ti-chi-bot bot pushed a commit that referenced this issue Feb 22, 2024
@ti-chi-bot
*: force_init_stats doesn't block http api's startup (#50853) (#51039)
0b8d947 
close #50854
@qiancai qiancai mentioned this issue Mar 6, 2024
Merged
18 tasks
ti-chi-bot bot pushed a commit that referenced this issue Mar 8, 2024
@ti-chi-bot
*: force_init_stats doesn't block http api's startup (#50853) (#51038)
fe48f7d 
close #50854
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.5 This bug affects the 6.5.x(LTS) versions. affects-7.1 This bug affects the 7.1.x(LTS) versions. affects-7.5 This bug affects the 7.5.x(LTS) versions. sig/planner SIG: Planner type/enhancement The issue or PR belongs to an enhancement.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

*: force_init_stats doesn't block http api's startup hawkingrei/tidb
3 participants
@hawkingrei @jebter @ti-chi-bot