hiero-ledger / hiero-ledger/hiero-consensus-node
Clean up Connection and Sync Input/Output stream classes
- 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
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