diff options
author | nagachika <nagachika@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2016-08-11 18:30:09 +0000 |
---|---|---|
committer | nagachika <nagachika@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2016-08-11 18:30:09 +0000 |
commit | c20b07d5357d7cb73226b149431a658cde54a697 (patch) | |
tree | 3e0e8711140765aa85f5019a3c7c5cc60081b2a6 | |
parent | 88d76cfdcd19bab721bfefadaa4f1c0de4e3c0ff (diff) |
merge revision(s) 55008: [Backport #12342]
* lib/drb/timeridconv.rb: don't use keeper thread. [Bug #12342]
* test/drb/ut_timerholder.rb: ditto.
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/branches/ruby_2_3@55867 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
-rw-r--r-- | ChangeLog | 6 | ||||
-rw-r--r-- | lib/drb/timeridconv.rb | 40 | ||||
-rw-r--r-- | test/drb/ut_timerholder.rb | 70 | ||||
-rw-r--r-- | version.h | 2 |
4 files changed, 51 insertions, 67 deletions
@@ -1,3 +1,9 @@ +Fri Aug 12 03:00:05 2016 Masatoshi SEKI <m_seki@mva.biglobe.ne.jp> + + * lib/drb/timeridconv.rb: don't use keeper thread. [Bug #12342] + + * test/drb/ut_timerholder.rb: ditto. + Fri Aug 12 02:46:37 2016 Kazuki Yamaguchi <k@rhe.jp> * ext/openssl/ossl_ssl.c (ossl_ssl_stop): Don't free the SSL struct diff --git a/lib/drb/timeridconv.rb b/lib/drb/timeridconv.rb index e83eb72765..7e2a6cf998 100644 --- a/lib/drb/timeridconv.rb +++ b/lib/drb/timeridconv.rb @@ -18,26 +18,27 @@ module DRb class InvalidIndexError < RuntimeError; end - def initialize(timeout=600) + def initialize(keeping=600) super() @sentinel = Object.new @gc = {} - @curr = {} @renew = {} - @timeout = timeout - @keeper = keeper + @keeping = keeping + @expires = Time.now + @keeping end def add(obj) synchronize do + rotate key = obj.__id__ - @curr[key] = obj + @renew[key] = obj return key end end def fetch(key, dv=@sentinel) synchronize do + rotate obj = peek(key) if obj == @sentinel return dv unless dv == @sentinel @@ -48,42 +49,35 @@ module DRb end end - def include?(key) - synchronize do - obj = peek(key) - return false if obj == @sentinel - true - end - end - + private def peek(key) synchronize do - return @curr.fetch(key, @renew.fetch(key, @gc.fetch(key, @sentinel))) + return @renew.fetch(key) { @gc.fetch(key, @sentinel) } end end - private - def alternate + def rotate synchronize do - @gc = @curr # GCed - @curr = @renew + return if @expires > Time.now + @gc = @renew # GCed @renew = {} + @expires = Time.now + @keeping end end def keeper Thread.new do loop do - alternate - sleep(@timeout) + rotate + sleep(@keeping) end end end end - # Creates a new TimerIdConv which will hold objects for +timeout+ seconds. - def initialize(timeout=600) - @holder = TimerHolder2.new(timeout) + # Creates a new TimerIdConv which will hold objects for +keeping+ seconds. + def initialize(keeping=600) + @holder = TimerHolder2.new(keeping) end def to_obj(ref) # :nodoc: diff --git a/test/drb/ut_timerholder.rb b/test/drb/ut_timerholder.rb index 1b992d95f3..252c7bd228 100644 --- a/test/drb/ut_timerholder.rb +++ b/test/drb/ut_timerholder.rb @@ -1,54 +1,38 @@ # frozen_string_literal: false -require 'runit/testcase' -require 'runit/cui/testrunner' -require 'timerholder' +require 'test/unit' +require 'drb/timeridconv' module DRbTests -class TimerHolderTest < RUNIT::TestCase - def do_test(timeout, keeper_sleep = nil) - holder = TimerHolder.new(timeout) - holder.keeper_sleep = keeper_sleep if keeper_sleep - key = holder.add(self) - sleep(timeout * 0.5) - assert_equal(holder.peek(key), self) - holder.delete(key) - assert(!holder.include?(key)) - key = holder.add(self) - sleep(timeout+0.5) - assert_equal(holder.fetch(key), nil) - key = holder.add(self) - assert_equal(holder.fetch(key), self) - holder.store(key, true) - assert_equal(holder.fetch(key), true) - assert_equal(holder.include?(key), true) - sleep(timeout+0.5) - assert_exception(TimerHolder::InvalidIndexError) do - holder.store(key, 1) - end - assert_equal(holder.include?(key), false) - key = holder.add(self) - sleep(timeout * 0.5) - assert(holder.include?(key)) - holder.extend(key, timeout) - sleep(timeout * 0.5) - assert(holder.include?(key)) - sleep(timeout * 0.6) - assert(!holder.include?(key)) - holder.delete(key) - end +class TimerIdConvTest < Test::Unit::TestCase + def test_usecase_01 + keeping = 0.1 + idconv = DRb::TimerIdConv.new(keeping) - def test_00 - do_test(0.5) - end + key = idconv.to_id(self) + assert_equal(key, self.__id__) + sleep(keeping) + + assert_equal(idconv.to_id(false), false.__id__) + assert_equal(idconv.to_obj(key), self) + sleep(keeping) + + assert_equal(idconv.to_obj(key), self) + sleep(keeping) + + assert_equal(idconv.to_id(true), true.__id__) + sleep(keeping) - def test_01 - do_test(1, 0.5) + assert_raise do + assert_equal(idconv.to_obj(key)) + end + + assert_raise do + assert_equal(idconv.to_obj(false.__id__)) + end end end -end -if __FILE__ == $0 - RUNIT::CUI::TestRunner.run(DRbTests::TimerHolderTest.suite) end + @@ -1,6 +1,6 @@ #define RUBY_VERSION "2.3.2" #define RUBY_RELEASE_DATE "2016-08-12" -#define RUBY_PATCHLEVEL 147 +#define RUBY_PATCHLEVEL 148 #define RUBY_RELEASE_YEAR 2016 #define RUBY_RELEASE_MONTH 8 |