fix: update data_substrate — prevent dirty_data_key_count_ underflow

[githubzilla]

Summary

Update data_substrate submodule to include fix for intermittent assertion failure:

Assertion `dirty_delta >= 0 || dirty_data_key_count_ >= static_cast<size_t>(-dirty_delta)' failed.

Three bugs fixed in tx_service (see eloqdata/tx_service#465 for full details):

1. CleanBucketData freeing entries being checkpointed — CanBeCleaned now checks !GetBeingCkpt() to prevent dirty count double-decrement (confirmed root cause via core dump + diagnostic logs)
2. BackFill wrong operation order — reordered SetCommitTsPayloadStatus before SetCkptTs to prevent flush bit from being cleared; added missing OnCommittedUpdate in BackFill and ReadOutsideCc paths
3. ClusterConfigCcMap missing OnCommittedUpdate — added dirty count tracking at both SetCommitTsPayloadStatus sites

Testing

Verified by running cluster_scale_test.py (all 5 tests) dozens of times with no assertion failures.

Summary by CodeRabbit

• Chores
  • Updated an internal submodule/dependency reference to a newer revision. No functional or public API changes; end-user behavior remains unchanged.

[coderabbitai]

Note

Reviews paused

It looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the reviews.auto_review.auto_pause_after_reviewed_commits setting.

Use the following commands to manage reviews:

• @coderabbitai resume to resume automatic reviews.
• @coderabbitai review to trigger a single review.

Use the checkboxes below for quick actions:

• ▶️ Resume reviews
• 🔍 Trigger review

Walkthrough

Updated the data_substrate git submodule pointer from ce06eee580f6bd35aaf1e8caed8b31286d11df59 to 807e7078303c3037c1bda92ee8865401e1d84cf5; no other code or API changes. Lines changed: +1 / -1.

Changes

Cohort / File(s)	Summary
Submodule Update ./.gitmodules, data_substrate	Changed data_substrate submodule checked-out commit from ce06eee580f6bd35aaf1e8caed8b31286d11df59 → 807e7078303c3037c1bda92ee8865401e1d84cf5 (metadata-only change).

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

• update submodule #356: Also updates the data_substrate submodule pointer (different target commit).
• chore: update submodule data_substrate #383: Submodule-pointer-only change for data_substrate.
• chore:update submodule data_substrate #435: Another PR that modifies the data_substrate submodule reference.

Suggested reviewers

• liunyl
• thweetkomputer

Poem

🐇 I nudged a hash beneath the log,
A tiny hop, a quiet cog,
The pointer moved, the code sleeps tight,
I twitch my whiskers in the night,
Repo dreams on in pale moonlight.

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 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 purpose of the PR: updating the data_substrate submodule to fix a specific bug related to dirty_data_key_count_ underflow.
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

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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