Skip to content

chore: update tx_service submodule for observability metrics - #563

Merged
liunyl merged 2 commits into
mainfrom
codex/cache-checkpoint-observability
Aug 30, 2026
Merged

chore: update tx_service submodule for observability metrics#563
liunyl merged 2 commits into
mainfrom
codex/cache-checkpoint-observability

Conversation

@liunyl

@liunyl liunyl commented Aug 29, 2026

Copy link
Copy Markdown
Contributor

Context

Parent integration for the cache and checkpoint observability implementation merged in eloqdata/tx_service#561.

Scope

This PR only updates the data_substrate submodule pointer. It intentionally excludes the glossary, ADR, implementation plan, temporary design documents, generated files, and local runtime artifacts.

Final dependency

The pointer targets tx_service main commit bf1dad98663608d42b6abe648a524bead83c09d0, the squash merge of tx_service#561. Its tree is identical to the previously CI-validated PR head 97ddec47a0ff072abec1cd65d37c3afde729e783.

Verification

  • TxService build: source EloqKV version from CMake #561 passed its required CI matrix and was approved before merge.
  • EloqKV full CI passed against the pre-squash dependency head.
  • The pre-squash and final merged TxService commits resolve to the same tree, so the submodule pointer replacement does not change built source content.
  • The final pointer is reachable from tx_service main.

Per delivery direction, the full EloqKV CI was not rerun for the pointer-only replacement.

Companion change

Rollback

Revert this submodule-pointer update.

Summary by CodeRabbit

  • Chores
    • Updated the underlying data substrate to the latest revision.
    • No user-facing behavior or functionality changes.

@coderabbitai

coderabbitai Bot commented Aug 29, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 50eec93e-67ac-4408-93e8-e855c281068d

📥 Commits

Reviewing files that changed from the base of the PR and between 45e416f and d262c00.

📒 Files selected for processing (1)
  • data_substrate
🚧 Files skipped from review as they are similar to previous changes (1)
  • data_substrate

Included review availability: Your plan provides up to 4 included reviews per hour; 3 remain after this review.


Walkthrough

The data_substrate subproject reference changed from commit b04468e79a4979a9770c7c7cf19cdae7bfaaa4f4 to commit bf1dad98663608d42b6abe648a524bead83c09d0.

Changes

Subproject update

Layer / File(s) Summary
Advance subproject reference
data_substrate
The pinned subproject commit hash changed to bf1dad98663608d42b6abe648a524bead83c09d0.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Merge Risk: ⚪ Minimal · up to d262c

This PR only updates the dependency pointer to an already CI-validated source tree, with no built-source change indicated; no actionable merge-blocking risk remains after normal checks and review.

Poem

A rabbit checks the pointer bright
One hash hops to new insight
No source code changes below
The substrate points where rabbits go
A tidy update, then off we flow

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Title check ✅ Passed The title clearly identifies the submodule update and its observability-metrics purpose.
Description check ✅ Passed The description is specific and relevant. It documents the context, scope, final dependency, verification, companion change, and rollback. It omits several template headings, but the available informa…
Full details: Docstring Coverage

Explanation

No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (1 skipped: 1 unsupported.)

Full details: Description check

Explanation

The description is specific and relevant. It documents the context, scope, final dependency, verification, companion change, and rollback. It omits several template headings, but the available information is sufficient for this pointer-only change.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch codex/cache-checkpoint-observability

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@liunyl
liunyl force-pushed the codex/cache-checkpoint-observability branch 2 times, most recently from a49d30a to abb0c19 Compare August 29, 2026 12:51
@liunyl liunyl changed the title feat(metrics): add cache and checkpoint observability chore: update tx_service submodule for observability metrics Aug 29, 2026
@liunyl
liunyl force-pushed the codex/cache-checkpoint-observability branch from abb0c19 to 45e416f Compare August 29, 2026 13:15
@liunyl
liunyl marked this pull request as ready for review August 29, 2026 13:32
@liunyl
liunyl requested a review from thweetkomputer August 30, 2026 02:25
@liunyl liunyl removed the trigger-ci label Aug 30, 2026
@liunyl
liunyl merged commit d8c4418 into main Aug 30, 2026
3 of 6 checks passed
@liunyl
liunyl deleted the codex/cache-checkpoint-observability branch August 30, 2026 15:01
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