Review Request 187: JobManager sync: [1/5] Synchronize JobManager with storage.

Michal Minar reviewboard at gallagherhome.com
Tue Apr 16 07:32:57 UTC 2013


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



src/python/openlmi/common/JobManager.py
<http://reviewboard-openlmi.rhcloud.com/r/187/#comment129>

    Try to avoid using % to format string for log messages. It enforces formatting before the log function is called and thus slows down the execution, when logging is suppressed.



src/python/openlmi/common/JobManager.py
<http://reviewboard-openlmi.rhcloud.com/r/187/#comment131>

    I would avoid this assignment. The default for path.keybindings is pywbem.NocaseDict, which does lowercase key comparisons. With this plain dictionary the whole path behaves differently and you have to bare it in mind, when using it afterwards.



src/python/openlmi/common/JobManager.py
<http://reviewboard-openlmi.rhcloud.com/r/187/#comment130>

    Maybe it's better to prevent logging by using different logger like:
      logging.getLogger('openlmi.separate_thread')
    
    which could have logging suppressed (runtime) or redirected to file (when debugging).


- Michal Minar


On April 12, 2013, 2:13 p.m., Jan Safranek wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard-openlmi.rhcloud.com/r/187/
> -----------------------------------------------------------
> 
> (Updated April 12, 2013, 2:13 p.m.)
> 
> 
> Review request for OpenLMI Developers.
> 
> 
> Repository: openlmi-providers
> 
> 
> Description
> -------
> 
> Synchronize JobManager with storage.
> 
> JobManager here is older version of the one in openlmi-storage.
> Let's sync it with the storage so I can use it from there.
> 
> The patch includes:
> 
> Fixed job expiration under SFCB.
> Added LMI_StorageJob.JobInParameters and .JobOutParameters properties.
> Allow python to exit the provider even if there are threads running.
> Fixed job returning an error
> Added possibility to set AffectedElements when a job finishes.
> Added workaround for rhbz#920763
> 
> 
> Diffs
> -----
> 
>   src/python/openlmi/common/JobManager.py a923f296d3eba40f1997a267b68525a44f50ffc7 
> 
> Diff: http://reviewboard-openlmi.rhcloud.com/r/187/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Jan Safranek
> 
>

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


More information about the openlmi-devel mailing list