Project

General

Profile

Feature #18940

Cache notification content to reduce server load

Added by Ohad Levy over 5 years ago. Updated almost 4 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Notifications
Target version:
Difficulty:
Triaged:
Bugzilla link:
Fixed in Releases:
Found in Releases:

Description

Since notification hit the server often (by default every 10 seconds when there is an active user session) it makes sense to cache the notification results.

Associated revisions

Revision d6a059c8 (diff)
Added by Ohad Levy about 5 years ago

fixes #18940 - adds caching to notification api calls.

this implementation reduce the notification fetching time on
my production system from avg response times of 80ms to 20ms,
and siginficantly reducing server load (db, cpu etc)

it also removes total counts from the implementation, as that was not
implemented in the UI at all.

History

#1 Updated by The Foreman Bot over 5 years ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/4383 added

#2 Updated by Ohad Levy about 5 years ago

  • Status changed from Ready For Testing to Closed
  • % Done changed from 0 to 100

#3 Updated by Ohad Levy about 5 years ago

  • Legacy Backlogs Release (now unused) set to 209

Also available in: Atom PDF