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

Achieving eventual consistency for statistics synchronization is unattainable. #54552

Open
hawkingrei opened this issue Jul 10, 2024 · 2 comments
Open

Achieving eventual consistency for statistics synchronization is unattainable. #54552

hawkingrei opened this issue Jul 10, 2024 · 2 comments
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. affects-8.1 This bug affects the 8.1.x(LTS) versions. affects-8.5 This bug affects the 8.5.x(LTS) versions. component/statistics report/community The community has encountered this bug. severity/major sig/planner SIG: Planner type/bug The issue is confirmed as a bug.

Comments

@hawkingrei
Copy link
Member

Bug Report

Please answer these questions before submitting your issue. Thanks!

1. Minimal reproduce step (Required)

When you have two tidb instance, if you run analyze on the one tidb.

2. What did you expect to see? (Required)

both of tidb can see this stats.

3. What did you see instead (Required)

the other tidb instance can be unable to see these stats.

4. What is your TiDB version? (Required)
@hawkingrei hawkingrei added the type/bug The issue is confirmed as a bug. label Jul 10, 2024
@jebter jebter added the sig/planner SIG: Planner label Jul 12, 2024
@ti-chi-bot ti-chi-bot bot added the affects-7.5 This bug affects the 7.5.x(LTS) versions. label Aug 30, 2024
@seiya-annie
Copy link

/report community

@ti-chi-bot ti-chi-bot bot added the report/community The community has encountered this bug. label Aug 30, 2024
@hawkingrei hawkingrei mentioned this issue Sep 9, 2024
Merged
13 tasks
@hawkingrei hawkingrei added 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-8.1 This bug affects the 8.1.x(LTS) versions. and removed may-affects-5.4 This bug maybe affects 5.4.x versions. may-affects-6.1 may-affects-6.5 may-affects-7.1 may-affects-8.1 labels Sep 9, 2024
ti-chi-bot bot pushed a commit that referenced this issue Sep 9, 2024
@hawkingrei
statstics: increase the synchronization offset size when to sync stats (
df1c21e 
#55943)

ref #54552
This was referenced Sep 9, 2024
Merged
Merged
Merged
Merged
ti-chi-bot bot pushed a commit that referenced this issue Sep 9, 2024
@ti-chi-bot
statstics: increase the synchronization offset size when to sync stats (
ef4a74a 
#55943) (#55960)

ref #54552
ti-chi-bot bot pushed a commit that referenced this issue Sep 12, 2024
@ti-chi-bot
statstics: increase the synchronization offset size when to sync stats (
e0bef39 
#55943) (#55963)

ref #54552
ti-chi-bot bot pushed a commit that referenced this issue Sep 13, 2024
@ti-chi-bot
statstics: increase the synchronization offset size when to sync stats (
20f729e 
#55943) (#55962)

ref #54552
@hawkingrei hawkingrei mentioned this issue Sep 25, 2024
Merged
13 tasks
ti-chi-bot bot pushed a commit that referenced this issue Sep 25, 2024
@hawkingrei
domain: move UpdateStatsHealthyMetrics into updateStatsWorker (#55386)
0d5e0e9 
ref #54552
This was referenced Sep 25, 2024
Open
Merged
Merged
Merged
ti-chi-bot bot pushed a commit that referenced this issue Oct 17, 2024
@ti-chi-bot
domain: move UpdateStatsHealthyMetrics into updateStatsWorker (#55386) (
9cb99bc 
#56326)

ref #54552
@hawkingrei
Copy link
Member Author

#56538

This somewhat aids in resolving synchronization inconsistencies.

It effectively accelerates the update of stats metadata. If that table is continuously updated, it will refresh every 5 minutes. Any inconsistency will be resolved within that interval, so waiting an additional 5 minutes should suffice.

@AilinKid AilinKid mentioned this issue Oct 21, 2024
Open
ti-chi-bot bot pushed a commit that referenced this issue Oct 28, 2024
@ti-chi-bot
domain: move UpdateStatsHealthyMetrics into updateStatsWorker (#55386) (
f54a797 
#56325)

ref #54552
@ti-chi-bot ti-chi-bot bot added the affects-8.5 This bug affects the 8.5.x(LTS) versions. label Nov 1, 2024
ti-chi-bot bot pushed a commit that referenced this issue Nov 4, 2024
@ti-chi-bot
statstics: increase the synchronization offset size when to sync stats (
d23d8a8 
#55943) (#55961)

ref #54552
ti-chi-bot bot pushed a commit that referenced this issue Nov 5, 2024
@ti-chi-bot
domain: move UpdateStatsHealthyMetrics into updateStatsWorker (#55386) (
dcdfba1 
#56324)

ref #54552
@Rustin170506 Rustin170506 moved this to 📋 Backlog in 😆Stats Dec 18, 2024
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. affects-8.1 This bug affects the 8.1.x(LTS) versions. affects-8.5 This bug affects the 8.5.x(LTS) versions. component/statistics report/community The community has encountered this bug. severity/major sig/planner SIG: Planner type/bug The issue is confirmed as a bug.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@hawkingrei @jebter @Rustin170506 @seiya-annie