-
Notifications
You must be signed in to change notification settings - Fork 2.2k
Closed
Closed
Copy link
Labels
affects-7.3affects-7.4affects-7.5This bug affects the 7.5.x(LTS) versions.This bug affects the 7.5.x(LTS) versions.affects-7.6affects-8.0affects-8.1This bug affects the 8.1.x(LTS) versions.This bug affects the 8.1.x(LTS) versions.affects-8.2affects-8.3affects-8.4affects-8.5This bug affects the 8.5.x(LTS) versions.This bug affects the 8.5.x(LTS) versions.severity/moderatesig/transactionSIG: TransactionSIG: Transactiontype/bugThe issue is confirmed as a bug.The issue is confirmed as a bug.
Description
Bug Report
tikv/components/resolved_ts/src/advance.rs
Lines 109 to 113 in 2614586
| // Ignore get tso errors since we will retry every `advdance_ts_interval`. | |
| let mut min_ts = pd_client.get_tso().await.unwrap_or_default(); | |
| if let Ok(mut last_pd_tso) = last_pd_tso.try_lock() { | |
| *last_pd_tso = Some((min_ts, Instant::now())); | |
| } |
last_pd_tso may be updated to 0.
The field is only used for diagnosis.
The consequence can be misleading gaps in logs and metrics, but not correctness risks.
What version of TiKV are you using?
master and versions in [v7.3, v8.5]
What operating system and CPU are you using?
Steps to reproduce
What did you expect?
What did happened?
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
affects-7.3affects-7.4affects-7.5This bug affects the 7.5.x(LTS) versions.This bug affects the 7.5.x(LTS) versions.affects-7.6affects-8.0affects-8.1This bug affects the 8.1.x(LTS) versions.This bug affects the 8.1.x(LTS) versions.affects-8.2affects-8.3affects-8.4affects-8.5This bug affects the 8.5.x(LTS) versions.This bug affects the 8.5.x(LTS) versions.severity/moderatesig/transactionSIG: TransactionSIG: Transactiontype/bugThe issue is confirmed as a bug.The issue is confirmed as a bug.