Jetty

Extend statistics to differentiate comet requests

Details

  • Type: New Feature New Feature
  • Status: Resolved Resolved
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: None
  • Fix Version/s: 7.0.0pre2
  • Component/s: Continuations
  • Labels:
    None
  • Number of attachments :
    4

Description

Extend the statistics handling to analyse comet requests so you can see requests that are being retried due to resumes/timeouts etc.

  1. File
    1-stats.diff
    09/Jun/08 10:34 PM
    20 kB
    Athena Yao
  2. File
    2-durationfix.diff
    09/Jun/08 10:34 PM
    9 kB
    Athena Yao
  3. File
    3-bandwidth.diff
    09/Jun/08 10:34 PM
    3 kB
    Athena Yao
  4. File
    p0.diff
    10/Jun/08 4:13 AM
    13 kB
    Athena Yao

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Athena Yao added a comment -

1-stats.diff is the updated StatisticsServlet output, plus tests to make life easier (though some of these feel shaky – still ran into timing issues despite setting up synchronized locks). There's also a bugfix in there for the previous version of the StatisticsHandler I submitted

2-durationfix.diff fixes the duration. Changing Suspendable.java seems invasive, but the only other way I can think of involves saving the per-request suspended/resumed times in the statistics handler itself. Thoughts?

3-bandwidth.diff totals up response body size in bytes

Show
Athena Yao added a comment - 1-stats.diff is the updated StatisticsServlet output, plus tests to make life easier (though some of these feel shaky – still ran into timing issues despite setting up synchronized locks). There's also a bugfix in there for the previous version of the StatisticsHandler I submitted 2-durationfix.diff fixes the duration. Changing Suspendable.java seems invasive, but the only other way I can think of involves saving the per-request suspended/resumed times in the statistics handler itself. Thoughts? 3-bandwidth.diff totals up response body size in bytes
Hide
Permalink
Athena Yao added a comment -

p0.diff - patch for statsreset, mbeans properties, added test harness. (The test harness currently fails on_statsHandler.getRequestsDurationTotal())

Show
Athena Yao added a comment - p0.diff - patch for statsreset, mbeans properties, added test harness. (The test harness currently fails on_statsHandler.getRequestsDurationTotal())

People

  • Assignee:
    Athena Yao
    Reporter:
    Jan Bartel
Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: