Skip to content
This repository has been archived by the owner on Jul 13, 2023. It is now read-only.

Reduce 503 occasions #378

Closed
bbangert opened this issue Feb 25, 2016 · 0 comments
Closed

Reduce 503 occasions #378

bbangert opened this issue Feb 25, 2016 · 0 comments
Assignees

Comments

@bbangert
Copy link
Member

When a user leaves, and a new notification comes in, we clear the node and return a 503 to try again. That can push a decent amount of work onto the app-server, it would likely be better to actually continue the logic in endpoint, such that:

  • clear the node entry
  • save the notification
  • new node lookup
  • tell node to fetch messages if there

So essentially all we'd do is remove the raise exception, and let the message be saved immediately. That would reduce this 503 path, without much of a db hit change since most of the time the user has left and we are going to be storing the message for awhile.

@bbangert bbangert added this to the PUSHSVC-0: quality milestone Feb 25, 2016
@bbangert bbangert added the ready label Feb 25, 2016
jrconlin added a commit that referenced this issue Mar 4, 2016
When a user is not connected, store the message and do not return a 503.

Closes #378
@jrconlin jrconlin added in progress and removed ready labels Mar 4, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants