You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Currently for the updated django entity that has an associated TriggerChannel the notification payload is sent via postgres NOTIFY and it contains the old and the new serialized entity state. On the listener the payload sent via postgres channel is used only to find the appropriate Notification record from the DB and then the payload from the record is used.
This sending of the complete payload is suboptimal:
the serialized size of the record that can be sent is limited by the max payload size (8k) that can be sent via the channel. Given that both old and new records are sent that size is rather small
the serialized payload makes roundtrips from postgres to application (in the NOTIFY message) and then back to server (as a query parameter)
The better solution seems to be to send only the notification ID for the TriggerChannel that has a persistent notifications.
The text was updated successfully, but these errors were encountered:
I have an implementation of this but it depends and is based on PRs that are not yet merged #66 and #73 and there might be some amendments to the approach there so I'll add a PR for this when it will be clear what migration path will be chosen there.
Currently for the updated django entity that has an associated
TriggerChannel
the notification payload is sent via postgresNOTIFY
and it contains the old and the new serialized entity state. On the listener the payload sent via postgres channel is used only to find the appropriateNotification
record from the DB and then the payload from the record is used.This sending of the complete payload is suboptimal:
old
andnew
records are sent that size is rather smallThe better solution seems to be to send only the notification ID for the
TriggerChannel
that has a persistent notifications.The text was updated successfully, but these errors were encountered: