From 0bffe57f784b40dc355c2eca82d1d7a8cab09093 Mon Sep 17 00:00:00 2001 From: Eric Wong Date: Tue, 1 Nov 2011 20:08:25 +0000 Subject: [PATCH] backend: rely on Array#shuffle instead sort_by { rand } This is easier-to-read and more correct. This is not available in 1.8.6, but we already broke 1.8.6 compatibility with String#bytesize. --- lib/mogilefs/backend.rb | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/mogilefs/backend.rb b/lib/mogilefs/backend.rb index 0ab2d4a..973d82b 100644 --- a/lib/mogilefs/backend.rb +++ b/lib/mogilefs/backend.rb @@ -221,7 +221,7 @@ class MogileFS::Backend now = Time.now - @hosts.sort_by { rand(3) - 1 }.each do |host| + @hosts.shuffle.each do |host| next if @dead.include? host and @dead[host] > now - 5 begin -- 2.11.4.GIT