WIP: Experiment: extract Searcher into an Actor #807

Closed

igalic wants to merge 2 commits from igalic/Plume:refactor/extract-searcher into main

igalic commented

Owner

The goal of this pull request is to remove Searcher from PlumeRocket, as described in #799.
The strategy in this pull request is to use a riker actor and

Wrap Searcher & DbPool into an actor
create messages for add, delete, update
initialize actor system in main()
manage "actor system, so we have easy access to it
RequestGuard on ActorSystem, to allow access to it
instead of using &PlumeRocket.Searcher, use ActorSystem to send a message to our SearcherActor

The goal of this pull request is to remove `Searcher` from `PlumeRocket`, as described in #799. The strategy in this pull request is to use a riker actor and - Wrap Searcher & DbPool into an actor - create messages for add, delete, update - initialize actor system in `main()` - manage "actor system, so we have easy access to it - RequestGuard on ActorSystem, to allow access to it - instead of using `&PlumeRocket.Searcher`, use ActorSystem to send a message to our SearcherActor

igalic added 2 commits

b59af45802

add riker as dependency, we will use it to replace searcher

ae4ec4f9bd

Add an SearcherActor, wrapping Searcher & DbPool

and implement AddDocument message!
This code compiles, but hasn't been put to use yet.

igalic referenced this pull request

[Refactoring] send messages to dedicated actor instead of directly accessing objects #799

igalic commented

Poster

Owner

closing in favour of #813

igalic closed this pull request

Please reopen this pull request to perform a merge.

No reviewers

No Milestone

No Assignees

1 Participants

Notifications

Due Date

The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Loading…

There is no content yet.

Labels Milestones

WIP: Experiment: extract Searcher into an Actor #807