Issue #4552 has been updated by Rein Henrichs.

Status changed from Unreviewed to Needs design decision

This feature could be implemented slightly differently to report timing 
information in milliseconds (which I imagine would be more useful than seconds) 
either by using Time.now.to_f (which gives fractional seconds) or by using the 
Benchmark standard library.

Also, I'm not thrilled by the use of the $timing and @@timing variables, nor by 
setting them to integral values rather than true/false, but I see that you were 
just following the pattern used for --debug, so I'll make my objections to 
those elsewhere :)
----------------------------------------
Feature #4552: Add Fact execution timing information
http://projects.puppetlabs.com/issues/4552

Author: Rein Henrichs
Status: Needs design decision
Priority: Normal
Assigned to: 
Category: 
Target version: 
Keywords: 
Branch: 


This patch from Dean Wilson adds a -t option to facter to show how long each 
fact is taking to execute.


-- 
You have received this notification because you have either subscribed to it, 
or are involved in it.
To change your notification preferences, please click here: 
http://projects.puppetlabs.com/my/account

-- 
You received this message because you are subscribed to the Google Groups 
"Puppet Bugs" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to 
[email protected].
For more options, visit this group at 
http://groups.google.com/group/puppet-bugs?hl=en.

Reply via email to