Skip to content

update submodules#198

Merged
MrGuin merged 1 commit intoeloqdata:mainfrom
MrGuin:ng_leader_term_cache
Sep 18, 2025
Merged

update submodules#198
MrGuin merged 1 commit intoeloqdata:mainfrom
MrGuin:ng_leader_term_cache

Conversation

@MrGuin
Copy link
Copy Markdown
Collaborator

@MrGuin MrGuin commented Sep 18, 2025

Summary by CodeRabbit

  • Chores
    • Updated an internal submodule reference to the latest upstream version to maintain alignment and stability.
    • No changes to user-facing features, behavior, or public APIs.
    • No configuration updates, migrations, or compatibility impacts required.
    • Performance, reliability, and functionality remain unchanged.

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Sep 18, 2025

Walkthrough

Submodule pointer for src/mongo/db/modules/eloq/tx_service was advanced from bd21f9a13c71c546699a443ca50f1a63d9097a57 to 48e5c862f36a0e5096c7e1bae42a1790b7c0fba7. No code or API changes within this repository are included.

Changes

Cohort / File(s) Summary of changes
Submodule update
src/mongo/db/modules/eloq/tx_service
Updated submodule reference from bd21f9a… to 48e5c8…; no in-repo source changes

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

  • update subm #189 — Updates the same tx_service submodule pointer to the prior commit; this PR advances it further.

Suggested labels

trigger-ci

Poem

A hop, a skip, a pointer new—
I nudge the submodule, onward I flew.
No code to chew, no API stew,
Just footprints fresh in version dew.
Thump-thump! CI, run it through. 🐇✨

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check ✅ Passed The title "update submodules" accurately reflects the primary change in the PR — a submodule pointer update (src/mongo/db/modules/eloq/tx_service) — and is concise and relevant to the changeset; it is somewhat generic but still directly related to the change.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.
✨ Finishing touches
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between b86b634 and 8bccb97.

📒 Files selected for processing (1)
  • src/mongo/db/modules/eloq/tx_service (1 hunks)
🔇 Additional comments (1)
src/mongo/db/modules/eloq/tx_service (1)

1-1: Submodule bump OK — 1-commit upgrade; confirm CI recursive submodule update

  • Path: src/mongo/db/modules/eloq/tx_service
  • URL: https://github.com/eloqdata/tx_service.git
  • Old: bd21f9a13c71c546699a443ca50f1a63d9097a57
  • New: 48e5c862f36a0e5096c7e1bae42a1790b7c0fba7
  • Change: upgrade — 1 commit: 48e5c86 "Store ng leader term with ng_leader_cache. ([Bug] EloqDoc hangs on jepsen test #129)"
  • Flags: no BREAKING / SECURITY / REVERT found
  • CI: ripgrep returned "No files were searched" when scanning .github/** for "submodule update" — cannot confirm CI uses recursive submodule checkout. Confirm workflows run git submodule update --init --recursive (or enable submodules in the checkout action).

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

@MrGuin MrGuin merged commit 099e7df into eloqdata:main Sep 18, 2025
2 checks passed
@MrGuin MrGuin deleted the ng_leader_term_cache branch September 18, 2025 10:07
xiexiaoy pushed a commit that referenced this pull request Sep 19, 2025
zhangh43 pushed a commit to zhangh43/eloqdoc that referenced this pull request Sep 28, 2025
This was referenced Oct 28, 2025
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