summaryrefslogtreecommitdiff
path: root/internal/processing/workers/workers.go
diff options
context:
space:
mode:
authorLibravatar kim <89579420+NyaaaWhatsUpDoc@users.noreply.github.com>2024-04-26 13:50:46 +0100
committerLibravatar GitHub <noreply@github.com>2024-04-26 13:50:46 +0100
commitc9c0773f2c2363dcfa37e675b83ec3f0b49bd0d9 (patch)
treedbd3409070765d5ca81448a574ccd32b4da1ffe6 /internal/processing/workers/workers.go
parent[chore] update Docker container to use new go swagger hash (#2872) (diff)
downloadgotosocial-c9c0773f2c2363dcfa37e675b83ec3f0b49bd0d9.tar.xz
[performance] update remaining worker pools to use queues (#2865)
* start replacing client + federator + media workers with new worker + queue types * refactor federatingDB.Delete(), drop queued messages when deleting account / status * move all queue purging to the processor workers * undo toolchain updates * code comments, ensure dereferencer worker pool gets started * update gruf libraries in readme * start the job scheduler separately to the worker pools * reshuffle ordering or server.go + remove duplicate worker start / stop * update go-list version * fix vendoring * move queue invalidation to before wipeing / deletion, to ensure queued work not dropped * add logging to worker processing functions in testrig, don't start workers in unexpected places * update go-structr to add (+then rely on) QueueCtx{} type * ensure more worker pools get started properly in tests * fix remaining broken tests relying on worker queue logic * fix account test suite queue popping logic, ensure noop workers do not pull from queue * move back accidentally shuffled account deletion order * ensure error (non nil!!) gets passed in refactored federatingDB{}.Delete() * silently drop deletes from accounts not permitted to * don't warn log on forwarded deletes * make if else clauses easier to parse * use getFederatorMsg() * improved code comment * improved code comment re: requesting account delete checks * remove boolean result from worker start / stop since false = already running or already stopped * remove optional passed-in http.client * remove worker starting from the admin CLI commands (we don't need to handle side-effects) * update prune cli to start scheduler but not all of the workers * fix rebase issues * remove redundant return statements * i'm sorry sir linter
Diffstat (limited to 'internal/processing/workers/workers.go')
-rw-r--r--internal/processing/workers/workers.go30
1 files changed, 15 insertions, 15 deletions
diff --git a/internal/processing/workers/workers.go b/internal/processing/workers/workers.go
index 8488e501c..1159b61a5 100644
--- a/internal/processing/workers/workers.go
+++ b/internal/processing/workers/workers.go
@@ -30,9 +30,9 @@ import (
)
type Processor struct {
+ clientAPI clientAPI
+ fediAPI fediAPI
workers *workers.Workers
- clientAPI *clientAPI
- fediAPI *fediAPI
}
func New(
@@ -45,6 +45,14 @@ func New(
media *media.Processor,
stream *stream.Processor,
) Processor {
+ // Init federate logic
+ // wrapper struct.
+ federate := &federate{
+ Federator: federator,
+ state: state,
+ converter: converter,
+ }
+
// Init surface logic
// wrapper struct.
surface := &surface{
@@ -55,16 +63,8 @@ func New(
emailSender: emailSender,
}
- // Init federate logic
- // wrapper struct.
- federate := &federate{
- Federator: federator,
- state: state,
- converter: converter,
- }
-
// Init shared util funcs.
- utilF := &utilF{
+ utils := &utils{
state: state,
media: media,
account: account,
@@ -73,20 +73,20 @@ func New(
return Processor{
workers: &state.Workers,
- clientAPI: &clientAPI{
+ clientAPI: clientAPI{
state: state,
converter: converter,
surface: surface,
federate: federate,
account: account,
- utilF: utilF,
+ utils: utils,
},
- fediAPI: &fediAPI{
+ fediAPI: fediAPI{
state: state,
surface: surface,
federate: federate,
account: account,
- utilF: utilF,
+ utils: utils,
},
}
}