Skip to content

update subm#189

Merged
liunyl merged 1 commit intoeloqdata:mainfrom
liunyl:batch_write
Sep 18, 2025
Merged

update subm#189
liunyl merged 1 commit intoeloqdata:mainfrom
liunyl:batch_write

Conversation

@liunyl
Copy link
Copy Markdown
Contributor

@liunyl liunyl commented Sep 17, 2025

Summary by CodeRabbit

  • Chores
    • Updated internal module references for storage handling and transaction services to newer commits.
    • No changes to features, user interface, public APIs, or workflows.
    • No impact on performance or compatibility.
    • No action required from users.

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Sep 17, 2025

Caution

Review failed

The pull request is closed.

Walkthrough

Updated submodule pointers for two Eloq components to new commit hashes. No source changes, control-flow modifications, or public API alterations are included.

Changes

Cohort / File(s) Change Summary
Eloq submodules
src/mongo/db/modules/eloq/store_handler, src/mongo/db/modules/eloq/tx_service
Updated submodule refs: store_handler 88d71f2 → ca901f6; tx_service 5b51fdf → bd21f9a. No code, control-flow, or API changes.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

  • update subm #189 — Updates the same eloq submodule pointers (store_handler and tx_service), closely related to this PR.

Poem

I twitch my whiskers at commits anew,
Two tiny hops, and the pointers flew.
No wires moved, no APIs stirred—
Just refs aligned, precise and blurred.
Thump-thump! I sign this merge with cheer. 🐇✨

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 subm" is related to the changeset because the PR updates submodule pointers for eloq/store_handler and eloq/tx_service, but it uses an unclear abbreviation ("subm") and lacks specificity about which submodules were changed.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.

📜 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 9ab8ccc and 3583f50.

📒 Files selected for processing (2)
  • src/mongo/db/modules/eloq/store_handler (1 hunks)
  • src/mongo/db/modules/eloq/tx_service (1 hunks)

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

@liunyl liunyl merged commit b86b634 into eloqdata:main Sep 18, 2025
1 of 2 checks passed
@liunyl liunyl deleted the batch_write branch September 18, 2025 06:47
zhangh43 pushed a commit to zhangh43/eloqdoc that referenced this pull request Sep 28, 2025
This was referenced Nov 21, 2025
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.

1 participant