Skip to content

update submodules#149

Merged
MrGuin merged 1 commit intoeloqdata:eloq-10.6.10from
MrGuin:fix_replay_max_ts_zx
Oct 28, 2025
Merged

update submodules#149
MrGuin merged 1 commit intoeloqdata:eloq-10.6.10from
MrGuin:fix_replay_max_ts_zx

Conversation

@MrGuin
Copy link
Collaborator

@MrGuin MrGuin commented Oct 27, 2025

Summary by CodeRabbit

  • Chores
    • Updated internal infrastructure components (submodules) to newer revisions; no user-facing or functional changes.

@coderabbitai
Copy link

coderabbitai bot commented Oct 27, 2025

Walkthrough

Submodule pointers for storage/eloq/store_handler and storage/eloq/tx_service were advanced to new commit hashes; no code or API changes were introduced and no functional logic was altered.

Changes

Cohort / File(s) Summary
Submodule pointer updates
storage/eloq/store_handler, storage/eloq/tx_service
Updated recorded commits: storage/eloq/store_handler from cd88ee6ec2a4684a8b032b75488319f923d888e7516fe3faaa991212abffb3fd01a5709a4698234d; storage/eloq/tx_service from a3cd3a013b07a9d9be2944aac5b549e7dd334c4e03d2d52bf103cee0f87f2bd7157d6b0d69f6f2e8. No functional changes.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

  • Metadata-only submodule pointer updates; no logic to inspect.

Possibly related PRs

Suggested reviewers

  • liunyl
  • thweetkomputer

Poem

🐰
Hop, hop — new commits land in line,
Tiny pointers shift, no code to pine.
I nibble metadata, tidy and spry,
Leaves unchanged, beneath the sky.

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. You can run @coderabbitai generate docstrings to improve docstring coverage.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check ✅ Passed The pull request title "update submodules" directly and accurately describes the primary change in the changeset. The raw summary indicates that both modified files (storage/eloq/store_handler and storage/eloq/tx_service) only have submodule pointer updates with no functional code changes. The title is concise, specific enough to convey the main purpose of the change, and would allow a teammate scanning history to quickly understand that this is a submodule version update. There is no vagueness, misleading information, or extraneous details.
✨ 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 25db4a6 and abfdbf4.

📒 Files selected for processing (2)
  • storage/eloq/store_handler (1 hunks)
  • storage/eloq/tx_service (1 hunks)
✅ Files skipped from review due to trivial changes (2)
  • storage/eloq/store_handler
  • storage/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

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

@MrGuin MrGuin force-pushed the fix_replay_max_ts_zx branch from 25db4a6 to abfdbf4 Compare October 28, 2025 09:17
@MrGuin MrGuin merged commit 1db805f into eloqdata:eloq-10.6.10 Oct 28, 2025
2 of 3 checks passed
@MrGuin MrGuin deleted the fix_replay_max_ts_zx branch October 28, 2025 10:42
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