Skip to content

Add term to eloqstore#430

Merged
lzxddz merged 1 commit intoeloqdata:mainfrom
lzxddz:eloqstore-multi-write-term
Jan 21, 2026
Merged

Add term to eloqstore#430
lzxddz merged 1 commit intoeloqdata:mainfrom
lzxddz:eloqstore-multi-write-term

Conversation

@lzxddz
Copy link
Copy Markdown
Collaborator

@lzxddz lzxddz commented Jan 17, 2026

Summary by CodeRabbit

  • Chores
    • Updated an internal repository reference to the latest revision to keep components in sync and maintain system stability.
    • Minor housekeeping: small metadata update with no user-facing API or behavior changes.

✏️ Tip: You can customize this high-level summary in your review settings.

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Jan 17, 2026

Walkthrough

Updates the submodule pointer for src/mongo/db/modules/eloq/data_substrate from commit 2fb175be7e3ad8fa294ced5c035989b915d3498c to cc1dc5b572bb9bf9f86310498a29ee59949e8f33; no source-code or public API changes.

Changes

Cohort / File(s) Summary
Submodule Pointer Update
src/mongo/db/modules/eloq/data_substrate
Submodule commit updated from 2fb175be7e3ad8fa294ced5c035989b915d3498c to cc1dc5b572bb9bf9f86310498a29ee59949e8f33 (lines changed: +1/-1).

Estimated Code Review Effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly Related PRs

  • update subm #381: Also updates the submodule pointer for src/mongo/db/modules/eloq/data_substrate.
  • fix bugs #417: Changes the same submodule pointer to a different commit hash.
  • update submodule #341: Modifies the tracked commit for src/mongo/db/modules/eloq/data_substrate.

Suggested Reviewers

  • liunyl
  • yi-xmu

Poem

🐰 A tiny hop, a pointer changed anew,

a hash replaced — just one line through.
No APIs bent, no sources stirred,
The substrate leaps with barely a word. 🥕

🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 inconclusive)
Check name Status Explanation Resolution
Title check ❓ Inconclusive The title 'Add term to eloqstore' is vague and does not clearly describe what term is being added or the actual purpose of updating the data_substrate submodule pointer. Revise the title to be more specific about what term is being added and why, such as 'Add multi-write term support to eloqstore' or similar descriptive language.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

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 and usage tips.

@lzxddz lzxddz force-pushed the eloqstore-multi-write-term branch 2 times, most recently from dd61149 to d151d1f Compare January 21, 2026 06:52
@lzxddz lzxddz force-pushed the eloqstore-multi-write-term branch from d151d1f to 3bb44c7 Compare January 21, 2026 09:32
@lzxddz lzxddz merged commit e1e6ebb into eloqdata:main Jan 21, 2026
2 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants