#4832 Add missing instrumentation for commit view pages

v1.0.0
closed
General
2015-08-20
2012-08-30
No

{"time": "2012-08-30 17:28:07,525", "level": "INFO ", "name": "stats", "message": {"url": "/p/slashmobdeploy/code/ci/6cac017cf4e182af52aa5af6853c16946876184b/", "uptime": 2710, "request_category": "git", "timings": {"total": 94767, "markdown": 1, "ming": 137, "jinja": 293, "mongo": 39}}}

This page takes a terribly long total time to render, but none of the individual timings are very high. We should have more instrumented functions to help identify what is taking so long.

Related

Tickets: #5625

Discussion

  • Dave Brondsema

    Dave Brondsema - 2012-10-18

    SVN too. {"time": "2012-10-17 08:10:30,681", "level": "INFO ", "name": "stats", "message": {"url": "/p/elec3609/code/9/", "uptime": 1473, "request_category": "svn", "timings": {"markdown": 1, "jinja": 50, "ming": 262, "socket_write": 0, "socket_read": 1149, "total": 473656, "mongo": 152}}}

     
  • Dave Brondsema

    Dave Brondsema - 2012-10-18
    • milestone: forge-backlog --> forge-nov-02
     
  • Dave Brondsema

    Dave Brondsema - 2012-11-02

    Filesystem IO would be my guess as to what we need to measure.

     
  • Dave Brondsema

    Dave Brondsema - 2012-11-02
    • Milestone: forge-nov-16 --> forge-nov-30
     
  • Dave Brondsema

    Dave Brondsema - 2012-11-16
    • milestone: forge-nov-30 --> forge-backlog
     
  • Dave Brondsema

    Dave Brondsema - 2013-01-02

    And HG:

    {"time": "2013-01-02 02:39:23,695", "level": "INFO ", "name": "stats", "message": {"url": "/p/enblend/code/ci/89d381db54667d8bde4d18bfb5000c38f52db6b5/", "uptime": 2953, "request_category": "hg", "timings": {"markdown": 4, "jinja": 260, "ming": 316, "socket_write": 0, "socket_read": 110, "total": 364588, "mongo": 78}}}
    
     
  • Dave Brondsema

    Dave Brondsema - 2013-01-03

    Adding a timing for the sidebar would be interesting. I don't think it's the main cause here, but listing all the branches & their commit counts could take some time.

     
  • Dave Brondsema

    Dave Brondsema - 2013-01-16
    • labels: performance --> performance, p2
     
  • Dave Brondsema

    Dave Brondsema - 2013-01-16

    Bumping to p2 since this SCM browse slowness is a broad problem

     
    • status: open --> in-progress
    • assigned_to: Tim Van Steenburgh
    • milestone: forge-backlog --> forge-feb-08
     
  • Dave Brondsema

    Dave Brondsema - 2013-01-28
    • size: --> 2
     
  • forge:tv/4832

    Also review HEAD on the timermiddleware repo.

    Siteops ticket for new package: https://trac.geek.net/trac/siteops/ticket/52840

     
    • status: in-progress --> code-review
     
  • Dave Brondsema

    Dave Brondsema - 2013-01-29
    • qa: Dave Brondsema
     
  • Dave Brondsema

    Dave Brondsema - 2013-01-29
    • status: code-review --> closed
     

Log in to post a comment.