Andrzej Wójcik (Tigase) opened 2 years ago
|
|
I've fixed found issues (described above) and created PR https://github.com/tigase/tigase-pubsub/pull/14 |
|
I revied the changes and they look ok. It seems PubSub got even more async with it :-) |
Type |
Bug
|
Priority |
Normal
|
Assignee | |
Version |
tigase-server-8.3.0
|
Issue Votes (0)
Watchers (0)
Sending the last published items blocks processing EventBus events as it is done on the EventBus threads and requires access to the database. Work requiring access to the database should be moved to a separate thread pool, but filtering (checking if anything needs to be done) should be left on the EventBus threads. That would reduce queue build-up, as EventBus will not be blocked and only filtered events will be queued on the new thread pool.
Additionally, presence from MUC should not be forwarded to PubSub (PEP) for processing, as there should be no need to process them, while they are creating artificial load on the PubSub component.