U4-7810 - DatabaseServerRegistrar should not execute as soon as a request is made, this slows down app start

Created by Shannon Deminick 21 Jan 2016, 17:35:11 Updated by Claus Jensen 25 Jan 2016, 09:37:21

When the DatabaseServerRegistrar is initialized with the first request (when a website starts), it will hit the db which is unnecessary. The db updates should be performed on a recurring background thread instead of waiting for successful requests and performing any database interaction in the same thread as the request. This can easily be moved to a RecurringTaskBase with a delay.

Comments

Shannon Deminick 21 Jan 2016, 17:36:11

This needs to be backported for this version,

  • rev: d86db6b2cd3ef511c74c8e6cb30ea4cc7fea03ef
  • rev: dfa45e70f4177cc292e0f406d1574b5bbad72aee


Shannon Deminick 22 Jan 2016, 16:06:18

I have backported now.


Claus Jensen 25 Jan 2016, 09:34:31

Looks good to me.


Claus Jensen 25 Jan 2016, 09:37:21

I've noticed an increased startup time (where the animated loading icon is showing for a long time compared to earlier) once in a while .. but have not been able to consistently reproduce it and not sure if it is even related to this change.

It's however not alarming and moving this to a separate background task seems like a good change overall.


Priority: Normal

Type: Performance Problem

State: Fixed

Assignee:

Difficulty: Normal

Category:

Backwards Compatible: True

Fix Submitted:

Affected versions:

Due in version: 7.3.6

Sprint: Sprint 7

Story Points:

Cycle: