Skip to content

refactor(db): move Pebble context under db common - #24

Merged
zhenghaoz merged 1 commit into
mainfrom
refactor/move-indexstore
Aug 15, 2026
Merged

refactor(db): move Pebble context under db common#24
zhenghaoz merged 1 commit into
mainfrom
refactor/move-indexstore

Conversation

@zhangzhenghao

@zhangzhenghao zhangzhenghao commented Aug 14, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • move the shared Pebble index-store wrapper from internal/indexstore/indexstore.go to internal/db/common/pebble_context.go
  • keep the Go package name as common
  • update FTS and inverted-index consumers without changing persistence behavior or disk markers

Verification

  • go test ./...
  • go test -race ./...
  • go vet ./...
  • git diff --check

@codecov

codecov Bot commented Aug 14, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 61.39%. Comparing base (a6dbfe2) to head (138e6a4).

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #24   +/-   ##
=======================================
  Coverage   61.39%   61.39%           
=======================================
  Files         155      155           
  Lines       40415    40415           
=======================================
  Hits        24812    24812           
  Misses      12366    12366           
  Partials     3237     3237           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@zhangzhenghao
zhangzhenghao force-pushed the refactor/move-indexstore branch from 3a7ec4a to 7d7870e Compare August 14, 2026 21:45
@zhangzhenghao
zhangzhenghao force-pushed the refactor/move-indexstore branch from 7d7870e to 138e6a4 Compare August 15, 2026 00:09
@zhenghaoz
zhenghaoz merged commit 236f506 into main Aug 15, 2026
7 checks passed
@zhenghaoz
zhenghaoz deleted the refactor/move-indexstore branch August 15, 2026 00:37
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