update submodule

[yi-xmu]

Summary by CodeRabbit

• Chores
  • Internal infrastructure update with no user-facing changes or functional impact.

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

[coderabbitai]

Walkthrough

A submodule pointer update for src/mongo/db/modules/eloq/data_substrate is being advanced from one commit hash to another. No functional changes or API modifications are included; only the recorded submodule commit reference has been updated.

Changes

Cohort / File(s)	Summary
Submodule Update src/mongo/db/modules/eloq/data_substrate	Pointer advanced from commit bc6852eec935a3ffca5918e540a544f38e9a9d2f to 1c1ad7699149a87a367a83a07cd3cb5e3efcef6b

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

• Update submodule for update datasyncscan execution #384: Updates the same submodule pointer with no functional changes.
• update submodule: fix for tx operation retry #399: Updates the same submodule pointer with no code/API changes.
• Update submodule #403: Updates the same submodule pointer with only commit-hash changes.

Suggested reviewers

• liunyl
• xiexiaoy

Poem

🐰 A hop and a skip, the submodule's on its way,
New commit hashes dancing, bringing updates to the day,
No breaking changes here, just a gentle little bump,
The data substrate flows with grace—our code takes quite the jump! 🌱

Pre-merge checks and finishing touches

❌ Failed checks (1 inconclusive)

Check name	Status	Explanation	Resolution
Title check	❓ Inconclusive	The title 'update submodule' is vague and generic, using non-descriptive language that fails to identify which submodule was updated or why.	Consider a more specific title like 'Update data_substrate submodule to 1c1ad76' to clearly indicate which submodule and what change was made.

✅ 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.

✨ Finishing touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch update_subm_ysw

---

📜 Recent review details

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between f82b030 and dfae9a4.

📒 Files selected for processing (1)

• src/mongo/db/modules/eloq/data_substrate

🧰 Additional context used

🧠 Learnings (2)

📓 Common learnings

Learnt from: githubzilla
Repo: eloqdata/eloqdoc PR: 279
File: src/mongo/db/modules/eloq/CMakeLists.txt:54-55
Timestamp: 2025-11-28T10:55:21.020Z
Learning: In src/mongo/db/modules/eloq/CMakeLists.txt, the CMake options EXT_TX_PROC_ENABLED, FORK_HM_PROCESS, and STATISTICS are used by the data_substrate submodule build system and should be retained as configuration interface options.

📚 Learning: 2025-11-28T10:55:21.020Z

Learnt from: githubzilla
Repo: eloqdata/eloqdoc PR: 279
File: src/mongo/db/modules/eloq/CMakeLists.txt:54-55
Timestamp: 2025-11-28T10:55:21.020Z
Learning: In src/mongo/db/modules/eloq/CMakeLists.txt, the CMake options EXT_TX_PROC_ENABLED, FORK_HM_PROCESS, and STATISTICS are used by the data_substrate submodule build system and should be retained as configuration interface options.

Applied to files:

• src/mongo/db/modules/eloq/data_substrate

🔇 Additional comments (1)

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

1-1: Verify submodule commit changes and breaking compatibility before merging.

The CMake options (EXT_TX_PROC_ENABLED, FORK_HM_PROCESS, STATISTICS) used by the data_substrate submodule are properly configured in the parent build system. However, before merging this submodule pointer update from bc6852ee... to 1c1ad76..., confirm:

1. Breaking changes in the submodule's API or build interface between these commits
2. Integration testing has been performed with the new submodule version
3. Backward compatibility is maintained for the CMake options the parent repo expects to pass to the submodule

---

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.}