search

tikv / raft-engine

A persistent storage engine for Multi-Raft log
Apache License 2.0
564 stars 88 forks source link

Fix `panics` on reading entries by stale index. #370

Closed LykxSassinator closed 1 week ago

LykxSassinator commented 1 week ago

What is changed and how it works?

Issue Number: Close #https://github.com/tikv/tikv/issues/17383

What's Changed:

To address the corner case where a read thread encounters a panic due to reading with a stale index from the Memtable, which has been updated by a background thread that has already purged the stale logs.

ti-chi-bot[bot] commented 1 week ago

Skipping CI for Draft Pull Request. If you want CI signal for your change, please convert it to an actual PR. You can still manually trigger a test run with /test all

wuhuizuo commented 1 week ago

/test ?

ti-chi-bot[bot] commented 1 week ago

@wuhuizuo: The following commands are available to trigger required jobs:

In response to [this](https://github.com/tikv/raft-engine/pull/370#issuecomment-2456603424): >/test ? Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md). If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes/test-infra](https://github.com/kubernetes/test-infra/issues/new?title=Prow%20issue:) repository.
wuhuizuo commented 1 week ago

/test rust-nightly

wuhuizuo commented 1 week ago

/test rust-stable

wuhuizuo commented 1 week ago

/test rust-nightly

ti-chi-bot[bot] commented 1 week ago

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: Connor1996, overvenus

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files: - ~~[OWNERS](https://github.com/tikv/raft-engine/blob/master/OWNERS)~~ [Connor1996,overvenus] Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment
ti-chi-bot[bot] commented 1 week ago

[LGTM Timeline notifier]

Timeline: