Skip to content

Comments

Convert deleted to put when enable mvcc#215

Merged
githubzilla merged 11 commits intoeloqdata:eloq-10.6.10from
githubzilla:delete_to_put
Jan 19, 2026
Merged

Convert deleted to put when enable mvcc#215
githubzilla merged 11 commits intoeloqdata:eloq-10.6.10from
githubzilla:delete_to_put

Conversation

@githubzilla
Copy link
Collaborator

@githubzilla githubzilla commented Jan 19, 2026

Summary by CodeRabbit

  • Chores
    • Updated an internal dependency pointer to a newer release. No functional or behavioral changes; no user-facing impact.
    • Review effort: low. Lines changed are minimal.

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

@coderabbitai
Copy link

coderabbitai bot commented Jan 19, 2026

Walkthrough

This pull request updates the data_substrate submodule pointer from commit c2b1c26101ba4bace22b14fe348a6d5d454f5d6d to abc0897ab94d99be96d6c2f4eab5d224489f22a9. No other code, configuration, or behavioral changes.

Changes

Cohort / File(s) Summary
Submodule Update
data_substrate
Updated submodule pointer from c2b1c26101ba4bace22b14fe348a6d5d454f5d6d to abc0897ab94d99be96d6c2f4eab5d224489f22a9 (lines changed: +1/-1)

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Suggested reviewers

  • lzxddz
  • xiexiaoy
  • liunyl

Poem

🐰 A tiny hop across the trees,
One pointer shifts with gentle ease,
From old hash nest to fresh new lair,
The substrate moves — a lighter air,
I nibble logs and hum with glee.

🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Title check ⚠️ Warning The PR title mentions 'Convert deleted to put when enable mvcc' but the changeset only updates a data_substrate submodule pointer with no visible behavioral changes related to the stated objective. Either update the title to accurately reflect that this is a submodule pointer update, or verify that the actual implementation of the mvcc conversion is present in the data_substrate commit being referenced.
✅ 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

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 merged commit b43d556 into eloqdata:eloq-10.6.10 Jan 19, 2026
2 checks passed
@coderabbitai coderabbitai bot mentioned this pull request Jan 20, 2026
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