puppetlabs/vmpooler
2
0
Fork 0
mirror of https://github.com/puppetlabs/vmpooler.git synced 2026-01-26 01:58:41 -05:00
Code Issues Projects Releases Packages Wiki Activity Actions

Merge pull request #66 from sschneid/hgetall_not_hget_in_a_loop

Browse source 
Use $redis.hgetall rather than hget in a loop
This commit is contained in:
Roger Ignazio 2015-03-09 11:43:19 -07:00
commit fa4147fb68
1 changed files with 2 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

4
lib/vmpooler/api/v1.rb
View file

@ -76,7 +76,7 @@ module Vmpooler
task[:count][:pool] = {} task[:count][:pool] = {}
task[:duration][:pool] = {} task[:duration][:pool] = {}
$redis.hkeys('vmpooler__' + task_str + '__' + date_str).each do |key| $redis.hgetall('vmpooler__' + task_str + '__' + date_str).each do |key, value|
pool = 'unknown' pool = 'unknown'
hostname = 'unknown' hostname = 'unknown'
@ -90,7 +90,7 @@ module Vmpooler
task[:duration][:pool][pool] ||= {} task[:duration][:pool][pool] ||= {}
task_times_bypool[pool] ||= [] task_times_bypool[pool] ||= []
task_times_bypool[pool].push($redis.hget('vmpooler__' + task_str + '__' + date_str, key).to_f) task_times_bypool[pool].push(value.to_f)
end end
task_times_bypool.each_key do |pool| task_times_bypool.each_key do |pool|