Review Request 204: Add TimerManager: [1/6] Add TimerManager.

Jan Safranek reviewboard at gallagherhome.com
Fri Apr 19 15:00:09 UTC 2013


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard-openlmi.rhcloud.com/r/204/
-----------------------------------------------------------

(Updated April 19, 2013, 5 p.m.)


Status
------

This change has been marked as submitted.


Review request for OpenLMI Developers.


Repository: openlmi-providers


Description
-------

Add TimerManager.

Standard Python threading.Timer starts separate thread for each timer.
That has two unpleseant consequences:
 - We end up with lot of sleeping threads.
 - If we want to use logging in thread callbacks, we must register the thread
   at CIMOM using AttachThread (=tedious).

So let's create TimerManager
 - Uses only on thread.
 - It is possible to log stuff in timer callbacks.


Diffs
-----

  src/python/openlmi/common/TimerManager.py PRE-CREATION 

Diff: http://reviewboard-openlmi.rhcloud.com/r/204/diff/


Testing
-------


Thanks,

Jan Safranek

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.fedorahosted.org/pipermail/openlmi-devel/attachments/20130419/d005198b/attachment.html>


More information about the openlmi-devel mailing list