#3142 (ok 3.4.0-alpha3) Status page takes 15 secs to generate

3.4.0
closed-fixed
None
1
2010-12-11
2010-11-26
No

The status page takes 15 secs to generate, that's way too long.
I also don't need any graphs in there.

Discussion

  • Dieter Adriaenssens
    Dieter Adriaenssens
    2010-12-04

    This bug was fixed in repository and will be part of a future release; thanks for reporting.

     
  • Dieter Adriaenssens
    Dieter Adriaenssens
    2010-12-04

    • priority: 5 --> 1
    • assigned_to: nobody --> ruleant
    • summary: Status page takes 15 secs to generate --> (ok 3.4.0-alpha3) Status page takes 15 secs to generate
    • status: open --> open-fixed
     
  • Olaf van der Spek
    Olaf van der Spek
    2010-12-04

    Just curious, what was the problem?

     
  • Dieter Adriaenssens
    Dieter Adriaenssens
    2010-12-04

    The newly added graph takes a long time to be generated.
    The fix doesn't load the graph by default, but provides a link to show (and thus generate) it.

     
  • Herman van Rink
    Herman van Rink
    2010-12-06

    When I reviewd the chart addotion I didn't notice such a delay.
    But since it is heavy making it optional seems like a good idea.

    The added warning seems a bit over the top though. Wouldn't that fit better in a title attribute?
    Or atleast change it to an PMA_Message::notice()

     
  • Dieter Adriaenssens
    Dieter Adriaenssens
    2010-12-06

    I got inspiration for the option+message from the database page where the statistics can be enabled/disabled, a warning is used there as well.
    But if you think it is too loud, it can be changed.

     
  • Herman van Rink
    Herman van Rink
    2010-12-06

    On the the database page the relation between the link and warning is stronger. (maybe it's the font-size or icon)
    At least they visually seem to belong more together.
    But it's probably on the loud side there as well.

    Lets ask the devel list...

     
  • Dieter Adriaenssens
    Dieter Adriaenssens
    2010-12-08

    Update : As discussed on the developers mailing list, the messages are now less 'loud' and the disable links are removed. Changes commited to latest git.

     
  • Marc Delisle
    Marc Delisle
    2010-12-11

    • status: open-fixed --> closed-fixed