polyphony 0.48.0 → 0.49.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 3a699d313072b1beba17c900491dd2a4f454bc8faea69eb157634e98cba7ee90
-  data.tar.gz: 3d8579b2c927c34876560ce878e960b9bc48a857ce43890dc069b4f0fa5ad75f
+  metadata.gz: 444ffbf8eb00cbd5e6948d6e3347e244fb8ea548aa1ad302d266e2f16c1a91b9
+  data.tar.gz: 2ac2f6ba1b51e71f1f138ad8eae665cc12e5458e836a0ef4f2df30d9ab95b679
 SHA512:
-  metadata.gz: d6efca5f42f21fe25ab5e6064675c2806dcad9d86e2e80c908987fcce855dd520ab51c4913e5a9001d3e4a84880a6606d124fa46dedff7c2e437a1ef6e156a82
-  data.tar.gz: 73ad6c59d4f7e9f8f0d317f480be1ba33c1a70dcf130a717693c2bb5a9ef3e226bfff3bdb635461b505bd2e9bbec7a2174c6a206bee2a94709ba9739ab087956
+  metadata.gz: 65924d13d191fc7c6d7b9fc2f7df997218396cf7ff9933564296ee50100441279387b01dc046797378973514dc45b25afd7eda92e19aac9dde3d7f2a13a9901e
+  data.tar.gz: a419d45a69c4a12a468277068ed792cc82d0384b2d620497c4c132fa27b9155ce05ce6662c062766ae5204c8c09a490bb1e95164c7a4bf0baa9f3166ac57d348

data/CHANGELOG.md

@@ -1,3 +1,7 @@
+## 0.49.0
+
+- Implement `Polyphony::Timer` for performant timeouts
+
 ## 0.48.0
 
 - Implement graceful shutdown

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    polyphony (0.48.0)
+    polyphony (0.49.0)
 
 GEM
   remote: https://rubygems.org/

data/LICENSE

@@ -1,6 +1,6 @@
 MIT License
 
-Copyright (c) 2020 Sharon Rosner
+Copyright (c) 2021 Sharon Rosner
 
 Permission is hereby granted, free of charge, to any person obtaining a copy
 of this software and associated documentation files (the "Software"), to deal

data/TODO.md

@@ -1,3 +1,5 @@
+- Check segfault when doing lots of `cancel_after` calls
+
 - Override stock `SizedQueue` impl with Queue with capacity
 
 - Add support for `break` and `StopIteration` in all loops (with tests)

data/lib/polyphony.rb

@@ -14,6 +14,7 @@ Thread.current.backend = Polyphony::Backend.new
 require_relative './polyphony/core/global_api'
 require_relative './polyphony/core/resource_pool'
 require_relative './polyphony/core/sync'
+require_relative './polyphony/core/timer'
 require_relative './polyphony/net'
 require_relative './polyphony/adapters/process'
 

data/lib/polyphony/core/timer.rb

@@ -0,0 +1,72 @@
+# frozen_string_literal: true
+
+module Polyphony
+  # Implements a common timer for running multiple timeouts
+  class Timer
+    def initialize(resolution:)
+      @fiber = spin_loop(interval: resolution) { update }
+      @timeouts = {}
+    end
+
+    def stop
+      @fiber.stop
+    end
+  
+    def cancel_after(duration, with_exception: Polyphony::Cancel)
+      fiber = Fiber.current
+      @timeouts[fiber] = {
+        duration: duration,
+        target_stamp: Time.now + duration,
+        exception: with_exception
+      }
+      yield
+    ensure
+      @timeouts.delete(fiber)
+    end
+
+    def move_on_after(duration, with_value: nil)
+      fiber = Fiber.current
+      @timeouts[fiber] = {
+        duration: duration,
+        target_stamp: Time.now + duration,
+        value: with_value
+      }
+      yield
+    rescue Polyphony::MoveOn => e
+      e.value
+    ensure
+      @timeouts.delete(fiber)
+    end
+  
+    def reset
+      record = @timeouts[Fiber.current]
+      return unless record
+  
+      record[:target_stamp] = Time.now + record[:duration]
+    end
+  
+    private
+
+    def timeout_exception(record)
+      case (exception = record[:exception])
+      when Class then exception.new
+      when Array then exception[0].new(exception[1])
+      when nil then Polyphony::MoveOn.new(record[:value])
+      else RuntimeError.new(exception)
+      end
+    end
+
+    def update
+      now = Time.now
+      # elapsed = nil
+      @timeouts.each do |fiber, record|
+        next if record[:target_stamp] > now
+
+        exception = timeout_exception(record)
+        # (elapsed ||= []) << fiber
+        fiber.schedule exception
+      end
+      # elapsed&.each { |f| @timeouts.delete(f) }
+    end
+  end
+end

data/lib/polyphony/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Polyphony
-  VERSION = '0.48.0'
+  VERSION = '0.49.0'
 end

data/test/test_fiber.rb

@@ -639,7 +639,6 @@ class FiberTest < MiniTest::Test
         i.close
         f.await
       rescue Exception => e
-        trace e
         o << e.class.name
         o.close
       end

data/test/test_socket.rb

@@ -63,14 +63,15 @@ class HTTPClientTest < MiniTest::Test
   require 'json'
 
   def test_http
-    res = HTTParty.get('http://worldtimeapi.org/api/timezone/Europe/Paris')
+    res = HTTParty.get('http://ipinfo.io/')
+
     response = JSON.load(res.body)
-    assert_equal "CET", response['abbreviation']
+    assert_equal 'https://ipinfo.io/missingauth', response['readme']
   end
 
   def test_https
-    res = HTTParty.get('https://worldtimeapi.org/api/timezone/Europe/Paris')
+    res = HTTParty.get('https://ipinfo.io/')
     response = JSON.load(res.body)
-    assert_equal "CET", response['abbreviation']
+    assert_equal 'https://ipinfo.io/missingauth', response['readme']
   end
 end

data/test/test_timer.rb

@@ -0,0 +1,122 @@
+# frozen_string_literal: true
+
+require_relative 'helper'
+
+class TimerMoveOnAfterTest < MiniTest::Test
+  def setup
+    @timer = Polyphony::Timer.new(resolution: 0.01)
+  end
+
+  def teardown
+    @timer.stop
+  end
+
+  def test_move_on_after
+    t0 = Time.now
+    v = @timer.move_on_after(0.1) do
+      sleep 1
+      :foo
+    end
+    t1 = Time.now
+
+    assert_in_range 0.1..0.15, t1 - t0
+    assert_nil v
+  end
+
+  def test_move_on_after_with_value
+    t0 = Time.now
+    v = @timer.move_on_after(0.01, with_value: :bar) do
+      sleep 1
+      :foo
+    end
+    t1 = Time.now
+
+    assert_in_range 0.01..0.02, t1 - t0
+    assert_equal :bar, v
+  end
+
+  def test_move_on_after_with_reset
+    t0 = Time.now
+    v = @timer.move_on_after(0.01, with_value: :moved_on) do
+      sleep 0.007
+      @timer.reset
+      sleep 0.007
+      nil
+    end
+    t1 = Time.now
+
+    assert_nil v
+    assert_in_range 0.014..0.02, t1 - t0
+  end
+end
+
+class CancelAfterTest < MiniTest::Test
+  def setup
+    @timer = Polyphony::Timer.new(resolution: 0.01)
+  end
+
+  def teardown
+    @timer.stop
+  end
+
+  def test_cancel_after
+    t0 = Time.now
+
+    assert_raises Polyphony::Cancel do
+      @timer.cancel_after(0.01) do
+        sleep 1
+        :foo
+      end
+    end
+    t1 = Time.now
+    assert_in_range 0.01..0.02, t1 - t0
+  end
+
+  def test_cancel_after_with_reset
+    t0 = Time.now
+    @timer.cancel_after(0.01) do
+      sleep 0.007
+      @timer.reset
+      sleep 0.007
+    end
+    t1 = Time.now
+    assert_in_range 0.014..0.024, t1 - t0
+  end
+
+  class CustomException < Exception
+  end
+
+  def test_cancel_after_with_custom_exception
+    assert_raises CustomException do
+      @timer.cancel_after(0.01, with_exception: CustomException) do
+        sleep 1
+        :foo
+      end
+    end
+
+    begin
+      err = nil
+      @timer.cancel_after(0.01, with_exception: [CustomException, 'custom message']) do
+        sleep 1
+        :foo
+      end
+    rescue Exception => err
+    ensure
+      assert_kind_of CustomException, err
+      assert_equal 'custom message', err.message
+    end
+
+
+    begin
+      e = nil
+      @timer.cancel_after(0.01, with_exception: 'foo') do
+        sleep 1
+        :foo
+      end
+    rescue => e
+    ensure
+      assert_kind_of RuntimeError, e
+      assert_equal 'foo', e.message
+    end
+  end
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: polyphony
 version: !ruby/object:Gem::Version
-  version: 0.48.0
+  version: 0.49.0
 platform: ruby
 authors:
 - Sharon Rosner
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2021-01-05 00:00:00.000000000 Z
+date: 2021-01-11 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake-compiler
@@ -476,6 +476,7 @@ files:
 - lib/polyphony/core/sync.rb
 - lib/polyphony/core/thread_pool.rb
 - lib/polyphony/core/throttler.rb
+- lib/polyphony/core/timer.rb
 - lib/polyphony/extensions/core.rb
 - lib/polyphony/extensions/debug.rb
 - lib/polyphony/extensions/fiber.rb
@@ -510,6 +511,7 @@ files:
 - test/test_thread.rb
 - test/test_thread_pool.rb
 - test/test_throttler.rb
+- test/test_timer.rb
 - test/test_trace.rb
 homepage: https://digital-fabric.github.io/polyphony
 licenses: