DWR

Examine the PollingServerLoadMonitor

Details

  • Type: Sub-task Sub-task
  • Status: In Progress In Progress
  • Priority: Normal Normal
  • Resolution: Unresolved
  • Affects Version/s: None
  • Fix Version/s: 3.0.1
  • Component/s: Reverse Ajax
  • Documentation Required:
    Yes
  • Description:
    Hide
    Matt the PollingServerLoadMonitor is another load monitor implementation that may need some work. Sorry I forgot about this monitor when I assigned you the other issues. For this issue take a look at it and decide what it needs to become more usable.

    This monitor is not currently being used. I have disabled it in trunk for concern over how the monitors in general were coded. It used to be enabled in BasePollHandler via:
    // Set the system up to resume anyway after maxConnectedTime
    ServerLoadMonitor slm = comet ? serverLoadMonitor : pollingServerLoadMonitor;

    Note: the comet variable has been renamed in the latest trunk version to - clientSupportsStreaming.
    Show
    Matt the PollingServerLoadMonitor is another load monitor implementation that may need some work. Sorry I forgot about this monitor when I assigned you the other issues. For this issue take a look at it and decide what it needs to become more usable. This monitor is not currently being used. I have disabled it in trunk for concern over how the monitors in general were coded. It used to be enabled in BasePollHandler via: // Set the system up to resume anyway after maxConnectedTime ServerLoadMonitor slm = comet ? serverLoadMonitor : pollingServerLoadMonitor; Note: the comet variable has been renamed in the latest trunk version to - clientSupportsStreaming.

Activity

Hide
Matt Conroy added a comment - 13/Jan/12 8:26 PM

I still need to dig into this one. It seems simple on the surface, but it could be better.

Show
Matt Conroy added a comment - 13/Jan/12 8:26 PM I still need to dig into this one. It seems simple on the surface, but it could be better.

People

Dates

  • Created:
    23/Apr/11 8:06 AM
    Updated:
    13/Jan/12 8:26 PM