Skip to content

update submodule#122

Merged
MrGuin merged 1 commit intoeloqdata:eloq-10.6.10from
MrGuin:ddl_log_ng_leader
Sep 23, 2025
Merged

update submodule#122
MrGuin merged 1 commit intoeloqdata:eloq-10.6.10from
MrGuin:ddl_log_ng_leader

Conversation

@MrGuin
Copy link
Collaborator

@MrGuin MrGuin commented Sep 19, 2025

Summary by CodeRabbit

  • Chores
    • Updated internal submodule references to track refreshed upstream states.
    • No changes to features, behavior, or user-facing interfaces.
    • Builds and deployments will fetch the updated submodule contents automatically.
    • No action required from users or administrators; usage remains unchanged.
    • Maintenance keeps dependencies current to support stability and compatibility.

@coderabbitai
Copy link

coderabbitai bot commented Sep 19, 2025

Warning

Rate limit exceeded

@MrGuin has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 19 minutes and 24 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

📥 Commits

Reviewing files that changed from the base of the PR and between c54eff8 and 5fc8e58.

📒 Files selected for processing (2)
  • storage/eloq/log_service (1 hunks)
  • storage/eloq/tx_service (1 hunks)

Walkthrough

The pull request updates submodule pointers for three storage/eloq submodules (tx_service, log_service, store_handler) to new commits. No source code, logic, APIs, or exported/public declarations in this repository were modified.

Changes

Cohort / File(s) Summary
Submodule pointer updates
storage/eloq/tx_service, storage/eloq/log_service, storage/eloq/store_handler
Submodule refs updated: storage/eloq/tx_service from 16f3cf1d6aabb763b1e8d2a71f0b06385a4631e3575b5609f555e4310a5a7a227b90fcac8e1103a7; storage/eloq/log_service from ee255c66aaee9e535d08a7ceedd34ade3c5f1f2beec3b2ba995097cb98d5e1543d241a454b2a7fcc; storage/eloq/store_handler from e0e4ef07c3a9cab621adc16c4edbf1f8935735db12155c653f1c876916ad21a2825d0e8e827c3f11. No direct code changes in this repo.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

  • update subm #119 — Updates the storage/eloq/store_handler submodule pointer (overlapping submodule change).
  • update subm #117 — Updates storage/eloq submodule pointers including store_handler and tx_service (directly related).

Poem

I nudged three pointers, soft and spry,
No code to scramble, tests pass by.
A tiny hop, new commits in tow,
The tree stays steady, CI aglow.
— Rabbit dev 🐇✨

Pre-merge checks and finishing touches

❌ Failed checks (1 inconclusive)
Check name Status Explanation Resolution
Title Check ❓ Inconclusive The title "update submodule" is related to the changes (the PR updates submodule pointers for storage/eloq/tx_service, storage/eloq/log_service, and storage/eloq/store_handler) but is overly generic and does not identify which submodules or the intent, so it does not meet the guideline for a clear, specific title. Please make the title specific and concise, for example "bump storage/eloq submodules: tx_service, log_service, store_handler" or "update storage/eloq submodule pointers to new commits" so reviewers can immediately see what changed.
✅ 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.

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

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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 6f1d0dd and 02965e4.

📒 Files selected for processing (1)
  • storage/eloq/tx_service (1 hunks)

@MrGuin MrGuin force-pushed the ddl_log_ng_leader branch 3 times, most recently from db3c312 to c54eff8 Compare September 23, 2025 05:39
@MrGuin MrGuin merged commit d0be810 into eloqdata:eloq-10.6.10 Sep 23, 2025
2 of 3 checks passed
@MrGuin MrGuin deleted the ddl_log_ng_leader branch September 23, 2025 06:05
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