Skip to content

InitCcm use table schema stored in local CatalogCcMap's CatalogRecord…#151

Open
MrGuin wants to merge 2 commits intoeloqdata:eloq-10.6.10from
MrGuin:fix_ccm_init
Open

InitCcm use table schema stored in local CatalogCcMap's CatalogRecord…#151
MrGuin wants to merge 2 commits intoeloqdata:eloq-10.6.10from
MrGuin:fix_ccm_init

Conversation

@MrGuin
Copy link
Collaborator

@MrGuin MrGuin commented Oct 30, 2025

… instead of LocalCcShards' CatalogEntry

Summary by CodeRabbit

  • Chores
    • Updated internal storage submodules to newer revisions; maintenance-only updates that improve reliability and internal behavior.
    • No visible UI, configuration, or user-facing behavioral changes. Operational or deployment processes may be affected only within internal systems.

@coderabbitai
Copy link

coderabbitai bot commented Oct 30, 2025

Walkthrough

Submodule pointers were updated for storage/eloq/tx_service and storage/eloq/store_handler to new target commits; no other code changes are indicated in this diff.

Changes

Cohort / File(s) Summary
Submodule version bump — tx_service
storage/eloq/tx_service
Updated submodule pointer from f3f89e6e6d9a4c8dca6ceed4b6058848ab256597 to 569c3628a7c61cc2b2954e54c02813ce72b3cc7c.
Submodule version bump — store_handler
storage/eloq/store_handler
Updated submodule pointer from 62e17d3b8c8c4095d094b66351fdd7dec3a4f4d4 to 8d8d1b3a9d08cc53b07910e2f80cf0002df2a2be.

Sequence Diagram(s)

(No sequence diagram — these are submodule pointer updates without control-flow or feature changes.)

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

  • Review focus:
    • Verify target commit hashes and that submodule .gitmodules/.gitlink updates are correct.
    • Inspect the referenced submodule commits for any behavioral changes affecting integration.
    • Confirm CI/integration tests and dependent modules build cleanly.

Possibly related PRs

Suggested reviewers

  • liunyl
  • lzxddz
  • thweetkomputer

Poem

🐰 I nudged two links with a gentle hop,
tx and store set to newer stop,
No code was touched, just pointers bright,
CI blinks once in the quiet night,
The rabbit twitches, then goes to crop.

Pre-merge checks and finishing touches

✅ Passed checks (2 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 InitCcm to use table schema from CatalogCcMap's CatalogRecord instead of LocalCcShards' CatalogEntry.
✨ 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 b7b9741 and a14bcf3.

📒 Files selected for processing (1)
  • storage/eloq/store_handler (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.

@MrGuin MrGuin force-pushed the fix_ccm_init branch 9 times, most recently from a205975 to 4ae6563 Compare November 4, 2025 06:54
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.

1 participant