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

Services need to be moved out of `app`: hinTS, history, roster, and block stream services

Open
#21,068 0 comments 0 reactions 1 assignee Claimed by @mhess-swl View on GitHub
Dominant language
Java
Stars
406
Forks
226
Avg merge
3d 4h
Merged PRs (30d)
210

Description

### Background

The current location of the hints, history, roster, and block stream services poses a problem when attempting to implement the CLPR prototype. CLPR, which lives in its own service module, needs access to the other services; but these other services live in the app module. This results in circular dependencies. Therefore, these services need to be refactored into their own respective modules.

(Note that it's possible to move CLPR into the app module as well, but we'd like to prevent this since services ideally live in independent modules anyway)

### Acceptance Criteria

All mentioned services are refactored to their own SPI and implementation modules

### Dependencies

We may need to put off moving the block stream service in order to meet the block node deliverable

Tasks:
- [ ] Roster — PR [here](https://github.com/hiero-ledger/hiero-consensus-node/pull/21202)
- [ ] Entity ID
- [ ] Block Stream
- [ ] Hints
- [ ] History

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.