Skip to content

feat: add support for compression with block block node connections - #26937

Draft
timfn-hg wants to merge 7 commits into
mainfrom
timfn/block-streaming-compression-2
Draft

feat: add support for compression with block block node connections#26937
timfn-hg wants to merge 7 commits into
mainfrom
timfn/block-streaming-compression-2

Conversation

@timfn-hg

Copy link
Copy Markdown
Contributor

Description:
Add support for compression when communicating with the block node

Related issue(s):

Fixes #

Notes for reviewer:

Checklist

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

Signed-off-by: Tim Farber-Newman <tim.farber-newman@swirldslabs.com>
Signed-off-by: Tim Farber-Newman <tim.farber-newman@swirldslabs.com>
@timfn-hg timfn-hg added this to the v0.79 milestone Aug 21, 2026
@timfn-hg timfn-hg self-assigned this Aug 21, 2026
@lfdt-bot

lfdt-bot commented Aug 21, 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.

@timfn-hg

timfn-hg commented Aug 21, 2026

Copy link
Copy Markdown
Contributor Author

Signed-off-by: Tim Farber-Newman <tim.farber-newman@swirldslabs.com>
@codecov

codecov Bot commented Aug 21, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 76.19048% with 5 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
.../blocks/impl/streaming/BlockNodeClientFactory.java 58.33% 4 Missing and 1 partial ⚠️

Impacted file tree graph

@@            Coverage Diff            @@
##               main   #26937   +/-   ##
=========================================
  Coverage     70.56%   70.56%           
  Complexity    11691    11691           
=========================================
  Files          2586     2587    +1     
  Lines        108566   108579   +13     
  Branches      12166    12168    +2     
=========================================
+ Hits          76605    76617   +12     
  Misses        27966    27966           
- Partials       3995     3996    +1     
Files with missing lines Coverage Δ Complexity Δ
...cks/impl/streaming/BlockNodeServiceConnection.java 90.82% <100.00%> (+0.17%) 0.00 <0.00> (ø)
...s/impl/streaming/BlockNodeStreamingConnection.java 80.18% <100.00%> (+0.04%) 0.00 <0.00> (ø)
...ra/node/config/data/BlockNodeConnectionConfig.java 100.00% <ø> (ø) 0.00 <0.00> (ø)
...e/config/types/BlockStreamGrpcCompressionType.java 100.00% <100.00%> (ø) 0.00 <0.00> (?)
.../blocks/impl/streaming/BlockNodeClientFactory.java 80.43% <58.33%> (-9.57%) 0.00 <0.00> (ø)

... and 1 file 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 28, 2026

Copy link
Copy Markdown

✅ All tests passed ✅

🏷️ Commit: 2cc91a6
▶️ Tests: 22236 executed
⚪️ Checks: 43/43 completed


Learn more about TestLens at testlens.app/docs.

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.

2 participants