eclipse-iceoryx / eclipse-iceoryx/iceoryx

ProcessManager refactoring

Open
#1,148 0 comments 0 reactions 0 assignees View on GitHub
technical debt
Dominant language
C++
Stars
2.2k
Forks
492
Avg merge
18h 57m
Merged PRs (30d)
1

Description

## Brief feature description

The `ProcessManager` does not only handle the processes but also handles the requests from the posh runtime. This makes is difficult to test and the process handling needs to be untangled.

## Detailed information

The code from `RouDi::processMessage` should be refactored to something like `CmdHandler`. This `CmdHandler` would take care of all the communication, receiving request and sending responses. The refactored `ProcessManager` would only take care of the process handling and be called by the `CmdHandler`.

This could be done in combination with #1133

Contributor guide

Open the contributing guide

Research direction

Start with RouDi::processMessage and trace how ProcessManager currently handles process work and requests from the posh runtime. Review the related scope in #1133 before deciding on the CmdHandler boundary. Done means communication is handled separately while ProcessManager focuses only on process handling, with tests made easier to isolate.

Written by the indexing model from the issue text.

Assessment

Tech stack
cpp
Domain
distributed-systems
Issue type
Refactor
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.