hiero-ledger / hiero-ledger/hiero-consensus-node
Revisit OutputWire.solderForMonitoring()
- Dominant language
- Java
- Stars
- 406
- Forks
- 226
- Avg merge
- 3d 4h
- Merged PRs (30d)
- 210
Description
The method `OutputWire.solderForMonitoring()` looks awkward. It seems to support only a single use case (hooking `EventPipelineTracker` into the system), and it is not obvious what the consequences are of adding a sink to the wiring, but not to the model. We should explore whether the same can be achieved without adding an extra method or whether the functionality can be generalized to make it reusable.
Contributor guide
Research direction
Start by locating OutputWire.solderForMonitoring() and all references to EventPipelineTracker. Trace how the monitoring sink is added to the wiring and how that differs from the model, then review the surrounding wiring APIs. Done means the team has a reusable or simpler design with its consequences understood and the single-use method addressed.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- java
- Domain
- backend
- Issue type
- Refactor
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Quiet
- Clarity
- Needs clarification
- Newbie friendliness
- 35/100