logstash-mixin-scheduler 1.0.0-java → 1.0.1-java

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 67978c6a1909fafc833c6b6ad0ea257274a7c6e9ae05dfd7c06e1a2b3d6820d0
-  data.tar.gz: cf0dada853778214e4bef783d356f921e0d153557912a6a00c787270decfaefe
+  metadata.gz: 7387c2af32713b46fd61936cfbe11854f3fdeaccb5c885d5acd0924eb69c3f66
+  data.tar.gz: cce64f89163bdd5a2b117e9fcfa7fb6fe61286bbe5156ed2572ea24147cee129
 SHA512:
-  metadata.gz: db8106c162a7268c40c3bfde3f7ee888d002238a9081f94eb294fa23af97eb687502a7243fd79b4e12c006574324bc1815b114966af43e42d73a837c7a4e3816
-  data.tar.gz: 0caadfd21d2b8e49f51b2b77cc3858423b0ec7fb21ba5aeab0839e2c17d308de34ecdcdea93e26e15cbc09b4bfccc435f5a05182e93a34eac5f69c3b019247a2
+  metadata.gz: 6da2594d0a2a09cba10bbac71bc675e51a48e1bd0f8f9be615f20f6db85ffebce72a5e325f0e0642f7c1f5511b683ee975256ac6a25649ab3181e78459d34dbe
+  data.tar.gz: 7b372d338d17aae260ff8a89ab2f5070d38e54ab50fa2d1f6e708dc2c77b16ad2f124714704d4f4fdca63bf036f0e69b4a4010c682a0f29d3095c69811da35a2

data/CHANGELOG.md

@@ -1,2 +1,5 @@
+## 1.0.1
+  - Refactor: initialize time zone data eagerly [#2](https://github.com/logstash-plugins/logstash-mixin-scheduler/pull/2)
+
 ## 1.0.0
   - Feat: a common scheduler interface [#1](https://github.com/logstash-plugins/logstash-mixin-scheduler/pull/1)

data/lib/logstash/plugin_mixins/scheduler/rufus_impl.rb

@@ -1,5 +1,11 @@
 require 'rufus/scheduler'
-
+begin
+  require 'et-orbi.rb' # a dependency of rufus-scheduler since 3.4
+  ::EtOrbi::EoTime.now # might take a long time to initialize - loading time zone
+  # data (from tz-info) and thus gets un-predictable on CI, since the scheduler worker
+  # thread might be stuck starting while we attempt to shutdown in a given time frame
+rescue LoadError
+end
 require 'logstash/util/loggable'
 
 module LogStash module PluginMixins module Scheduler module RufusImpl

data/spec/logstash/plugin_mixins/scheduler/rufus_impl_spec.rb

@@ -7,7 +7,7 @@ describe LogStash::PluginMixins::Scheduler::RufusImpl do
   let(:name) { '[test]<jdbc_scheduler' }
 
   let(:opts) do
-    { :max_work_threads => 2 }
+    { max_work_threads: 2, frequency: 0.2 }
   end
 
   subject(:scheduler) do
@@ -83,7 +83,10 @@ describe LogStash::PluginMixins::Scheduler::RufusImpl do
     let(:counter) { java.util.concurrent.atomic.AtomicLong.new(0) }
 
     before do
-      scheduler.cron('* * * * * *') { counter.increment_and_get; sleep 3.25 } # every second
+      scheduler.cron('* * * * * *') do # every second
+        counter.increment_and_get
+        sleep_at_least 3.25
+      end
     end
 
     it "are working" do
@@ -102,4 +105,17 @@ describe LogStash::PluginMixins::Scheduler::RufusImpl do
 
   end
 
+  private
+
+  # with multiple threads on JRuby 9.3 -> some tend to finish 'early'
+  def sleep_at_least(time)
+    start = Time.now
+    slept = 0
+    while slept < time
+      sleep(time - slept)
+      slept = Time.now - start
+    end
+    slept
+  end
+
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: logstash-mixin-scheduler
 version: !ruby/object:Gem::Version
-  version: 1.0.0
+  version: 1.0.1
 platform: java
 authors:
 - Elastic
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2022-06-07 00:00:00.000000000 Z
+date: 2022-06-14 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   requirement: !ruby/object:Gem::Requirement