Fix build

[lokax]

Summary by CodeRabbit

• Refactor

  • Replaced an internal scanner implementation with a more specialized variant; public interfaces, behavior, and user-visible outcomes remain unchanged.

• Chores

  • Updated internal submodule references; no changes to functionality, outputs, performance expectations, or required user actions.

[coderabbitai]

Walkthrough

Replaced the concrete scanner type returned by CreateRangeCcmScanner to txservice::HashParitionCcScanner<MongoKey, txservice::RangeRecord>; updated store_handler and tx_service submodule pointers. No signature, call-site, control-flow, or behavioral changes detected. (≤50 words)

Changes

Cohort / File(s)	Summary
Scanner type substitution src/mongo/db/modules/eloq/src/base/eloq_catalog_factory.cpp	Replaced returned scanner implementation from txservice::TemplateCcScanner<MongoKey, txservice::RangeRecord> to txservice::HashParitionCcScanner<MongoKey, txservice::RangeRecord> in CreateRangeCcmScanner; interface and parameters unchanged.
Submodule pointer updates src/mongo/db/modules/eloq/store_handler/*, src/mongo/db/modules/eloq/tx_service/*	Updated store_handler and tx_service submodule commit pointers to newer commits; no code-level changes in this diff.

Sequence Diagram(s)

(Skipped — change is a type substitution without control-flow or interaction changes.)

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

• Adjust hash partition mapping in Eloq submodules #199 — Switches CreateRangeCcmScanner to construct the hash-partition scanner and updates related submodules, closely related to this change.

Suggested reviewers

• xiexiaoy
• thweetkomputer
• liunyl

Poem

I nibble code beneath the log,
A hashed hop replaces the bog.
Quiet swap, no panic, no clatter,
New scanner hums — the bytes still matter.
Rabbit winks and carries on, happy as ever. 🐇

Pre-merge checks and finishing touches

❌ Failed checks (1 inconclusive)

Check name	Status	Explanation	Resolution
Title Check	❓ Inconclusive	The title “Fix build” is overly generic and does not specify the main change of replacing the TemplateCcScanner with the HashPartitionCcScanner in CreateRangeCcmScanner, so it lacks clarity for reviewers and history.	Please revise the title to clearly summarize the key change, for example “Replace TemplateCcScanner with HashPartitionCcScanner in CreateRangeCcmScanner to fix build.”

✅ 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 changes. Docstring coverage check skipped.

✨ Finishing touches

🧪 Generate unit tests (beta)

• 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 7d234f2 and 1b304fd.

📒 Files selected for processing (1)

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

✅ Files skipped from review due to trivial changes (1)

• src/mongo/db/modules/eloq/tx_service

---

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 fd24035 and 88dd481.

📒 Files selected for processing (3)

• src/mongo/db/modules/eloq/src/base/eloq_catalog_factory.cpp (1 hunks)
• src/mongo/db/modules/eloq/store_handler (1 hunks)
• src/mongo/db/modules/eloq/tx_service (1 hunks)

🚧 Files skipped from review as they are similar to previous changes (1)

• src/mongo/db/modules/eloq/src/base/eloq_catalog_factory.cpp

[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 b115564 and 7d234f2.

📒 Files selected for processing (2)

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

✅ Files skipped from review due to trivial changes (1)

• src/mongo/db/modules/eloq/tx_service

[coderabbitai]

⚠️ Potential issue | 🔴 Critical

❓ Verification inconclusive

Please confirm the submodule commit exists.

Before merging, make sure commit 48a5685ac196eefb73c0b78b51cf1d32a67d5f02 is present in the store_handler remote so the build doesn’t break when cloning.

---

🏁 Script executed:

#!/bin/bash
# Verify the submodule commit is reachable from its remote
cd src/mongo/db/modules/eloq/store_handler
git fetch --all --tags
git cat-file -t 48a5685ac196eefb73c0b78b51cf1d32a67d5f02

Length of output: 347

---

Submodule commit missing from remote
Commit 48a5685ac196eefb73c0b78b51cf1d32a67d5f02 isn’t present in the eloqdoc remote. Push that commit or update the submodule pointer to a valid, existing commit before merging.

[yi-xmu]

+1