Skip to content

Update submodule for txservice#161

Merged
xiexiaoy merged 1 commit intoeloqdata:eloq-10.6.10from
xiexiaoy:reset_deadlock_rsp
Nov 12, 2025
Merged

Update submodule for txservice#161
xiexiaoy merged 1 commit intoeloqdata:eloq-10.6.10from
xiexiaoy:reset_deadlock_rsp

Conversation

@xiexiaoy
Copy link
Collaborator

@xiexiaoy xiexiaoy commented Nov 12, 2025

eloqdata/tx_service#207

Summary by CodeRabbit

  • Chores
    • Updated internal submodule references to newer commits for storage components; no functional or public API changes.

@coderabbitai
Copy link

coderabbitai bot commented Nov 12, 2025

Caution

Review failed

The pull request is closed.

Walkthrough

Updates two git submodule pointers under storage/eloq: tx_service advanced to a new commit, and store_handler advanced to a new commit. No code, configuration, or exported/public entity declarations were changed.

Changes

Cohort / File(s) Change Summary
Submodule: tx_service
storage/eloq/tx_service
Updated submodule commit pointer from 123d7f7ab1af0d19e341ba590d9163994ab20838 to b5f974df8a9d66dcb29436376105d59adc6521da.
Submodule: store_handler
storage/eloq/store_handler
Updated submodule commit pointer from ce0e9317fd486a0e5a1342228ca4f2805d492daf to 979e159476542b1f0f7e8519d91457529ed0ad22.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

  • Metadata-only pointer updates; no code logic to review.

Possibly related PRs

Suggested reviewers

  • lzxddz
  • thweetkomputer

Poem

🐰 I hopped through commits, light and spry,
Two pointers nudged beneath the sky,
No code was harmed, just hashes new,
A tidy hop — that's all I do! 🎉

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The pull request title directly and accurately describes the main change: updating a submodule reference for txservice.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

📜 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 6d6136d and 3b5a7ee.

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

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.

@xiexiaoy xiexiaoy marked this pull request as ready for review November 12, 2025 06:11
@xiexiaoy xiexiaoy merged commit 3addf15 into eloqdata:eloq-10.6.10 Nov 12, 2025
1 of 2 checks passed
@xiexiaoy xiexiaoy deleted the reset_deadlock_rsp branch November 12, 2025 08:40
This was referenced Nov 24, 2025
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