Skip to content

[seekdb][sanity] Fix coredump in plan/PS cache and skip-index output - #1319

Merged
hnwyllmm merged 3 commits into
masterfrom
issue/2026081900118350907
Aug 20, 2026
Merged

[seekdb][sanity] Fix coredump in plan/PS cache and skip-index output#1319
hnwyllmm merged 3 commits into
masterfrom
issue/2026081900118350907

Conversation

@hnwyllmm

Copy link
Copy Markdown
Member

Task Description

Sanity Lite regression identified coredumps related to the seekdb source code: Plan Cache/PS Cache had object lifecycle issues during concurrent eviction and memory statistics, and the skip-index column output had an error in using the local buffer length.

Solution Description

  1. Complete cache object accounting and refresh within the Plan Cache node write lock scope to avoid accessing the object after releasing the last node reference.
  2. PS Cache statistics callback now uses a snapshot of the item/info size to avoid dereferencing a destroyed ps item after the schema expires.
  3. The skip-index column output delimiter now uses the actual extra print buffer capacity to avoid false positives in Sanity boundary checks and prevent abort.
  4. Added regression coverage for Plan Cache memory accounting lifecycle.

Passed Regressions

git diff --check passed. The fixed version has passed the previous Sanity Lite long-term regression, and the aforementioned coredumps have not reappeared. The local Bazel compilation of the current branch is blocked by an existing baseline issue with the repository's source/header inventory; this unrelated issue has not been modified.

Upgrade Compatibility

No changes to persistent formats, protocols, or upgrade compatibility are involved.

Other Information

This MR contains two commits: Plan/PS Cache lifecycle fixes and skip-index buffer length fixes. The Sanity PageArena detector issue itself is not included in this MR.

Release Note

@hnwyllmm
hnwyllmm merged commit e66776e into oceanbase:master Aug 20, 2026
1 check passed
@hnwyllmm
hnwyllmm deleted the issue/2026081900118350907 branch August 20, 2026 03:14
@hnwyllmm

Copy link
Copy Markdown
Member Author

The mapping Dima issue is: [SeekDB] [Sanity] Lite regression found a coredump caused by the seekdb source code.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants