Skip to content

chore: update submodule data_substrate#453

Merged
thweetkomputer merged 11 commits intoeloqdata:mainfrom
thweetkomputer:update-submodule-zc
Feb 9, 2026
Merged

chore: update submodule data_substrate#453
thweetkomputer merged 11 commits intoeloqdata:mainfrom
thweetkomputer:update-submodule-zc

Conversation

@thweetkomputer
Copy link
Collaborator

@thweetkomputer thweetkomputer commented Feb 3, 2026

Summary by CodeRabbit

  • Chores

    • Updated an internal submodule to a newer revision.
  • Chores / Configuration

    • Increased the store's open-files limit and added a pages-per-file setting to adjust storage layout.
  • Notes

    • No observable user-facing changes; behavior, features, and public interfaces remain unchanged.

@coderabbitai
Copy link

coderabbitai bot commented Feb 3, 2026

Note

Reviews paused

It looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the reviews.auto_review.auto_pause_after_reviewed_commits setting.

Use the following commands to manage reviews:

  • @coderabbitai resume to resume automatic reviews.
  • @coderabbitai review to trigger a single review.
  • ▶️ Resume reviews
  • 🔍 Trigger review

Walkthrough

Updated the src/mongo/db/modules/eloq/data_substrate submodule pointer and changed [store] settings in concourse/scripts/data_substrate.cnf: increased eloq_store_open_files_limit and added eloq_store_pages_per_file_shift. No exported-entity or API changes. (30 words)

Changes

Cohort / File(s) Summary
Submodule Pointer
src/mongo/db/modules/eloq/data_substrate
Bumped tracked submodule commit pointer (pointer-only change; no source/API/behavior modifications).
Concourse config
concourse/scripts/data_substrate.cnf
In [store]: eloq_store_open_files_limit increased from 4096 to 40960; added eloq_store_pages_per_file_shift=11.

Sequence Diagram(s)

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Suggested reviewers

  • liunyl
  • yi-xmu

Poem

🐰 I hopped through branches, quick and spry,
I nudged a pointer as the commits went by,
Gave file handles breadth and shifted a page,
A tiny config tweak to steady the stage,
I munch on logs and bounce up high.

🚥 Pre-merge checks | ✅ 3
✅ 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 accurately describes the main change: updating the data_substrate submodule pointer. However, it omits the equally important configuration changes to data_substrate.cnf.
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

No actionable comments were generated in the recent review. 🎉


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.

@thweetkomputer thweetkomputer changed the title chore: update submodule eloqstore: fix droptable chore: update submodule data_substrate Feb 9, 2026
@thweetkomputer thweetkomputer merged commit 6eed853 into eloqdata:main Feb 9, 2026
2 checks passed
@thweetkomputer thweetkomputer deleted the update-submodule-zc branch February 9, 2026 02:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants