This is an automatically generated e-mail. To reply, visit: http://reviewboard-openlmi.rhcloud.com/r/699/ |
On August 8th, 2013, 12:49 p.m. UTC, Jan Safranek wrote:
cli/lmi/shell/LMIUtil.py (Diff revision 1) 269 Helper function, which temporarily disables LMIShell's cache and returns CIMClass
270 with no properties, qualifiers to save network communication.
Why is caching disabled here? Maybe some superclass is already loaded. And it would be nice to have separate(?) cache for these 'minimal' classes, lmi_isinstance() can be used quite a lot.On August 8th, 2013, 12:50 p.m. UTC, Jan Safranek wrote:
Just an idea, you don't need to cache whole class, just some dictionary class->superclass.
Already working on that. I will cache {class : superclass}. Is that for you?
- Peter
On August 8th, 2013, 12:40 p.m. UTC, Peter Hatina wrote:
Review request for OpenLMI Developers.
By Peter Hatina.
Updated Aug. 8, 2013, 12:40 p.m.
Repository:
openlmi-tools
Description
Diffs
|