Skip to content

chore: Truncate wrapped-record-hashes.pb file and enable writing of hashes (#26747) - #26833

Open
derektriley wants to merge 2 commits into
release/0.78from
rel78-truncate-wrb
Open

chore: Truncate wrapped-record-hashes.pb file and enable writing of hashes (#26747)#26833
derektriley wants to merge 2 commits into
release/0.78from
rel78-truncate-wrb

Conversation

@derektriley

Copy link
Copy Markdown
Contributor

Description:
This PR cherry-picks 85ab0cc for release/0.78

Related issue(s):

Fixes #26746

Notes for reviewer:

Checklist

  • Documented (Code comments, README, etc.)
  • Tested (unit, integration, etc.)

Signed-off-by: Derek Riley <derek.riley@swirldslabs.com>
@derektriley derektriley added this to the v0.78 milestone Aug 13, 2026
@derektriley derektriley self-assigned this Aug 13, 2026
@derektriley
derektriley requested review from a team as code owners August 13, 2026 12:32
@lfdt-bot

lfdt-bot commented Aug 13, 2026

Copy link
Copy Markdown

Snyk checks have passed. No issues have been found so far.

Status Scan Engine Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

petreze
petreze previously approved these changes Aug 13, 2026
Signed-off-by: Derek Riley <derek.riley@swirldslabs.com>
@derektriley derektriley changed the title chore: Truncate wrapped-record-hashes.pb file (#26747) chore: Truncate wrapped-record-hashes.pb file and enable writing of hashes (#26747) Aug 13, 2026
@swirlds-automation
swirlds-automation dismissed petreze’s stale review August 13, 2026 12:39

1 file(s) changed in commit e8c2f28

@codecov

codecov Bot commented Aug 13, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 78.94737% with 4 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
.../records/impl/WrappedRecordBlockHashMigration.java 78.94% 3 Missing and 1 partial ⚠️

Impacted file tree graph

@@                Coverage Diff                 @@
##             release/0.78   #26833      +/-   ##
==================================================
+ Coverage           70.46%   70.47%   +0.01%     
+ Complexity          11675    11674       -1     
==================================================
  Files                2580     2580              
  Lines              108259   108278      +19     
  Branches            12095    12097       +2     
==================================================
+ Hits                76286    76314      +28     
+ Misses              27997    27984      -13     
- Partials             3976     3980       +4     
Files with missing lines Coverage Δ Complexity Δ
...dera/node/config/data/BlockRecordStreamConfig.java 100.00% <ø> (ø) 0.00 <0.00> (ø)
.../records/impl/WrappedRecordBlockHashMigration.java 87.15% <78.94%> (-0.79%) 0.00 <0.00> (ø)

... and 5 files with indirect coverage changes

Impacted file tree graph

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@testlens-app

testlens-app Bot commented Aug 13, 2026

Copy link
Copy Markdown

✅ All tests passed ✅

🏷️ Commit: e8c2f28
▶️ Tests: 4398 executed
⚪️ Checks: 46/46 completed


Learn more about TestLens at testlens.app.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants