[Unicode]   Common Locale Data Repository : Bug Tracking Home | Site Map | Search
 
Modify

CLDR Ticket #6993(accepted unknown)

Opened 4 years ago

Last modified 3 years ago

Use Timers for certain periodic tasks in the Webapp

Reported by: ribnitz Owned by: srl
Component: perf Data Locale:
Phase: dsub Review:
Weeks: Data Xpath:
Xref:

Description

Certain classes in the WebApp go to great lengths to call "cleanup" methods (CookieSession.checkForExpiredSessions is a good example for this).

It would probably be cleaner to have a Runnable that is called periodically, which does this work.

Attachments

Change History

comment:1 Changed 4 years ago by emmons

  • Owner changed from anybody to ribnitz
  • Milestone changed from UNSCH to 25final

comment:2 Changed 4 years ago by mark

  • Milestone changed from 25final to 26dsub

comment:3 Changed 3 years ago by emmons

  • Milestone changed from 26dsub to 26dvet

Moving all 26dsub to 26dvet. Please assess the need to complete tickets by 26dvet, which is 2014-06-19

comment:4 Changed 3 years ago by mark

  • Priority changed from assess to minor

comment:5 Changed 3 years ago by mark

  • Owner changed from ribnitz to googler

comment:6 Changed 3 years ago by ribnitz

The method checkForExpiredSessions() performs a number of tasks, that need to be run periodically. Ultimately it returns the number of users, and updates the number of guests and the number of users.

The idea is to modify both variables (nUsers, nGuests) to be volatile, and have checkForExpiredSessions simply return one of them. The bulk of the work would be in a new Runnable, which is then scheduled to be run periodically.

If the JSP container does not provide a Timer or ThreadPool where this can be scheduled, one can be created easily:

ScheduledExecutorService es= Executors.newScheduledExecutorService(...);

es.schedule(...);

comment:7 Changed 3 years ago by mark

  • Owner changed from googler to anybody
  • Milestone changed from 26dvet to 27dsub

comment:8 Changed 3 years ago by markus

  • Phase set to dsub
  • Milestone changed from 27dsub to 27

comment:9 Changed 3 years ago by emmons

  • Component changed from unknown to perf

comment:10 Changed 3 years ago by emmons

  • Milestone changed from 27 to UNSCH

Moving all 27+anybody tickets to UNSCH.

comment:11 Changed 3 years ago by emmons

  • Owner changed from anybody to srl
  • Status changed from new to assigned

comment:12 Changed 2 years ago by srl

  • Status changed from assigned to accepted
View

Add a comment

Modify Ticket

Action
as accepted
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.