update submodules

[MrGuin]

Summary by CodeRabbit

• Chores
  • Updated internal module dependencies to latest versions. No user-facing changes in this release.

[coderabbitai]

Walkthrough

Two eloq submodule pointer references are bumped to newer commits. The store_handler submodule is updated to commit ce0e9317, and the tx_service submodule is updated to commit 307387086a. Both are metadata-only changes with no functional code alterations.

Changes

Cohort / File(s)	Summary
Eloq submodule updates src/mongo/db/modules/eloq/store_handler, src/mongo/db/modules/eloq/tx_service	Submodule pointer references bumped to new commits (0f1c124 → ce0e931 for store_handler; 876d8b9 → 3073870 for tx_service)

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

• Pointer-only updates with no code changes require minimal verification

Possibly related PRs

• update submodules #277: Updates the same eloq submodule pointers (store_handler and tx_service) with no functional code changes
• update submodules #228: Updates submodule pointers for the same modules (store_handler and tx_service) with no code changes
• Fix slow open rocksdb cloud when DB is large #200: Modifies the same store_handler submodule pointer reference

Suggested reviewers

• liunyl
• thweetkomputer

Poem

🐰 Two pointers hop to new commits bright,
Store_handler and tx_service take flight!
No code to parse, just refs to update,
A tiny change that feels just great! 🌱

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 describes the main change: updating two submodule references in the pull request.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✨ 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

• X
• Mastodon
• Reddit
• LinkedIn

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

[coderabbitai]

Actionable comments posted: 1

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between fc80a8f and c3e4c19.

📒 Files selected for processing (2)

• src/mongo/db/modules/eloq/store_handler (1 hunks)
• src/mongo/db/modules/eloq/tx_service (1 hunks)

🔇 Additional comments (1)

src/mongo/db/modules/eloq/store_handler (1)

1-1: Verify the submodule update and provide changelog context.

This updates the store_handler submodule pointer from 0f1c1243db27802ff96087b1bc868b37010f8511 to ce0e9317fd486a0e5a1342228ca4f2805d492daf. The PR title references updating submodules, and the branch name fix_reader_writer_cntl suggests a specific fix, but there is no description of what these commits contain or why they are necessary.

Please confirm the following before merge:

1. Submodule contents: What functional changes or fixes do these new commits introduce? Provide links to commits in the submodule repositories (store_handler and tx_service).
2. Compatibility: Are there any breaking changes, API modifications, or behavioral changes that downstream consumers should be aware of?
3. Testing: Have these submodule updates been validated locally and via CI? The PR has a trigger-ci label—confirm CI has passed.
4. Coordination: The enriched summary mentions both store_handler and tx_service are updated. Are these updates coupled, or can they be deployed independently?

[coderabbitai]

⚠️ Potential issue | 🔴 Critical

🧩 Analysis chain

Verify the submodule commits and document the purpose of the updates.

Submodule pointer updates require verification of the target commits and clarity on what changes are being introduced. The branch name fix_reader_writer_cntl suggests a specific fix, but the PR description doesn't document what changed in the tx_service and store_handler submodules.

Please verify and document:

1. Commit validity: Confirm both commits (tx_service: 307387086a..., store_handler: ce0e9317...) are valid, reachable, and tested in their respective repositories.

2. Change summary: Provide a summary of what changed in each submodule commit:

   • What bugs or issues are fixed?
   • Are there breaking changes?
   • Any security implications?

3. Alignment with PR objectives: Explain how these submodule updates relate to the fix_reader_writer_cntl branch name and the overall PR goal.

You can run the following verification script to inspect the commits (adjust repository URLs as needed):

---

🏁 Script executed:

#!/bin/bash
# Description: Verify submodule commit references are valid and fetch commit details

# Assuming you have the submodule repos cloned or can access them via git
# Example: verify tx_service commit
git log --oneline -1 307387086a3e05d9da8c31dc6b818923361b076b 2>/dev/null || echo "Commit 307387086a not found in local history"

# If you have access to the remote repos, you can also check:
# git ls-remote <tx_service_repo_url> 307387086a3e05d9da8c31dc6b818923361b076b
# git ls-remote <store_handler_repo_url> ce0e9317fd486a0e5a1342228ca4f2805d492daf

Length of output: 44

---

🏁 Script executed:

#!/bin/bash
# Find and inspect .gitmodules to get submodule repository URLs
if [ -f .gitmodules ]; then
    echo "=== .gitmodules content ==="
    cat .gitmodules
    echo ""
    echo "=== Extracting submodule URLs ==="
    grep -E "^\[submodule|url = " .gitmodules
else
    echo ".gitmodules not found"
fi

Length of output: 1072

---

🏁 Script executed:

#!/bin/bash
# Verify submodule commit hashes using git ls-remote
echo "=== Verifying tx_service commit: 307387086a3e05d9da8c31dc6b818923361b076b ==="
git ls-remote https://github.com/eloqdata/tx_service.git 307387086a3e05d9da8c31dc6b818923361b076b || echo "Commit not found or repo unreachable"

echo ""
echo "=== Verifying store_handler commit: ce0e9317fd486a0e5a1342228ca4f2805d492daf ==="
git ls-remote https://github.com/eloqdata/store_handler.git ce0e9317fd486a0e5a1342228ca4f2805d492daf || echo "Commit not found or repo unreachable"

# Also try to get recent commit info to understand if repos are accessible
echo ""
echo "=== Recent commits in tx_service ==="
git ls-remote --heads --tags https://github.com/eloqdata/tx_service.git | head -10

echo ""
echo "=== Recent commits in store_handler ==="
git ls-remote --heads --tags https://github.com/eloqdata/store_handler.git | head -10

Length of output: 1645

---

Critical: Submodule commits cannot be verified in remote repositories.

The target commits do not exist or are not reachable in the remote repositories:

• tx_service commit 307387086a3e05d9da8c31dc6b818923361b076b: not found
• store_handler commit ce0e9317fd486a0e5a1342228ca4f2805d492daf: not found

Both repositories are accessible, but the specified commits are absent from all visible refs (branches and tags). This blocks merging the PR and poses a risk of submodule initialization failures.

Required actions:

1. Verify the commit hashes are correct
2. Confirm the commits have been pushed to the remote repositories (https://github.com/eloqdata/tx_service.git and https://github.com/eloqdata/store_handler.git)
3. Ensure commits are reachable from at least one branch or tag
4. Update the submodule pointers to valid, verified commits before merging

🤖 Prompt for AI Agents

In src/mongo/db/modules/eloq/tx_service around lines 1 to 1, the PR references
submodule commits that are not present in the remote repos (tx_service
307387086a3e05d9da8c31dc6b818923361b076b and store_handler
ce0e9317fd486a0e5a1342228ca4f2805d492daf); verify the commit hashes locally and
on the remotes, push the missing commits to their respective repositories
(https://github.com/eloqdata/tx_service.git and
https://github.com/eloqdata/store_handler.git), ensure each commit is reachable
from at least one branch or tag (or create a tag/branch pointing to it), and
then update the submodule pointers in this repo (.gitmodules and the submodule
SHA in the tree) to reference the verified, reachable commit SHAs before
merging.