Skip to content

Add log_retention_seconds#388

Merged
githubzilla merged 2 commits intoeloqdata:mainfrom
githubzilla:log_retention_seconds
Jan 12, 2026
Merged

Add log_retention_seconds#388
githubzilla merged 2 commits intoeloqdata:mainfrom
githubzilla:log_retention_seconds

Conversation

@githubzilla
Copy link
Copy Markdown
Collaborator

@githubzilla githubzilla commented Dec 21, 2025

Summary by CodeRabbit

  • Chores
    • Updated an internal submodule reference. This is an internal maintenance change with no user-facing behavior, performance, or public API modifications; no functional code changes are included.

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

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Dec 21, 2025

Caution

Review failed

The pull request is closed.

Walkthrough

Submodule pointer src/mongo/db/modules/eloq/data_substrate advanced from commit 203c8ce0c34264ec174a45188a01a3393f995aad to a43c2d2b911debf4967d90c3a4e5d893e2ac3c0d. No functional code changes were introduced.

Changes

Cohort / File(s) Summary
Submodule pointer update
src/mongo/db/modules/eloq/data_substrate
Updated submodule reference from 203c8ce0c34264ec174a45188a01a3393f995aada43c2d2b911debf4967d90c3a4e5d893e2ac3c0d (pointer-only update; no code changes)

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Suggested reviewers

  • xiexiaoy

Poem

🐇 I nudged the SHA, a tiny hop,
The forest stays still, I didn't stop.
No code disturbed, the branches hum,
One small move — then off I run. 🥕

🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Title check ⚠️ Warning The pull request title 'Add log_retention_seconds' does not match the actual changeset, which only updates a submodule pointer in src/mongo/db/modules/eloq/data_substrate with no functional code changes related to log retention. Update the title to accurately reflect the submodule update, such as 'Update eloq/data_substrate submodule pointer' or 'Update data_substrate submodule to latest commit'.
✅ 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.

✏️ 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

📜 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 c153396 and 13acbb9.

📒 Files selected for processing (1)
  • src/mongo/db/modules/eloq/data_substrate

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.

@githubzilla githubzilla force-pushed the log_retention_seconds branch from 1833537 to c153396 Compare January 11, 2026 13:05
@xiexiaoy xiexiaoy self-requested a review January 12, 2026 03:44
@githubzilla githubzilla merged commit 0f44ebb into eloqdata:main Jan 12, 2026
1 of 2 checks passed
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.

2 participants