hiero-ledger / hiero-ledger/hiero-consensus-node

Clean up Connection and Sync Input/Output stream classes

Open
#11,899 0 comments 0 reactions 0 assignees View on GitHub
Platform
Dominant language
Java
Stars
406
Forks
226
Avg merge
3d 4h
Merged PRs (30d)
210

Description

The classes `Connection` and `SyncInputSream`/`SyncOutputStream` have a bunch of methods that contain business logic for sync. This is not a good place for that logic to live. We should transfer the sync specific logic into helper static methods and colocate those methods with the sync code.

Contributor guide

Open the contributing guide

Research direction

Locate the Connection, SyncInputSream, and SyncOutputStream classes and identify the methods containing synchronization business logic. Trace the existing sync code to determine where helper static methods should be colocated; done means the sync-specific logic has moved out of the stream classes without changing their behavior.

Written by the indexing model from the issue text.

Assessment

Tech stack
java
Domain
networking
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.