Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Refactor camel code #1645

Closed
brosenberg42 opened this issue Feb 15, 2023 · 0 comments
Closed

Refactor camel code #1645

brosenberg42 opened this issue Feb 15, 2023 · 0 comments
Assignees
Milestone

Comments

@brosenberg42
Copy link
Member

brosenberg42 commented Feb 15, 2023

Various aspects of how Workflow Manager uses Camel can be improved.

  • Camel's default split and aggregate functionality can be used in multiple places instead of the custom CountBasedWfmAggregator. In places where custom split and aggregate functionality is required, it can be simplified by making better use of exchange properties Camel automatically sets.
  • Each of the processors in DetectionResponseRouteBuilder get and store the same tracks in Redis. Change it so that the tracks are only retrieved and stored once for the entire route.
  • TaskAggregationRouteBuilder just uses a single processor so it doesn't need to be its own route.
@brosenberg42 brosenberg42 added this to the Milestone 3 milestone Feb 15, 2023
@brosenberg42 brosenberg42 self-assigned this Feb 15, 2023
@brosenberg42 brosenberg42 added this to To do in OpenMPF: Development via automation Feb 15, 2023
@clnowacki clnowacki moved this from To do to In Progress in OpenMPF: Development Apr 13, 2023
@jrobble jrobble closed this as completed Jun 21, 2023
OpenMPF: Development automation moved this from In Progress to Closed Jun 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Development

No branches or pull requests

2 participants