logstash-filter-translate 3.3.1 → 3.4.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 77bd945dee5e20dbc7eb6837c655e807373ee8923dcbd498c9fcf5b1065c6661
-  data.tar.gz: 878df6d8dfcc5fa53bb617163b0ddfe6feeb8e876aed9668b337f66dc8472221
+  metadata.gz: f628dd5414571a8ad48ac804294f9404b4eaa460457e15955824d8a2ff5b6809
+  data.tar.gz: 6e762d49f1030f379fb5033c75c177f4eca134672bb446a27bb7c69821858f03
 SHA512:
-  metadata.gz: d45ec30865d73884f9564f3680fdc1fb230c9fa71507c86d475e6ec0759d13863080f8c237ab34b38ccad08bca0781ab1d8a8523955841db2d0a0334091a9cf3
-  data.tar.gz: 443284f8db270ec6081a2017bd6597733be886643232d97a7630c99e1b194970183e0a7d070ed511bdb1d8069e7f929a991d608798a35d8bd5408878806945d5
+  metadata.gz: d8a2a50b3668fa320d10587fd903d0a47a84b5cb0416876aea7ebdfad73a0692379c4e34b60aaa3b49a6297a7deb418ce87bbcc40947a85c673af549b00bc358
+  data.tar.gz: bc28c2a3e20d1b8f2b5d3c713978e32e5fee1ed2ea95d74af446f5a686096ee6ae7e179cf828aba5c2857f187a1449b61ab22e0d19e16fbaf5e1c8e540971471

data/CHANGELOG.md

@@ -1,3 +1,6 @@
+## 3.4.0
+ - Refactor: leverage scheduler mixin [#93](https://github.com/logstash-plugins/logstash-filter-translate/pull/93)
+
 ## 3.3.1
   - Refactor: reading .csv for JRuby 9.3 compatibility [#94](https://github.com/logstash-plugins/logstash-filter-translate/pull/94)
 

data/lib/logstash/filters/dictionary/file.rb

@@ -1,15 +1,14 @@
 # encoding: utf-8
-require 'concurrent/atomic/atomic_boolean'
-require 'rufus-scheduler'
 require "logstash/util/loggable"
 require "logstash/filters/fetch_strategy/file"
 
-java_import 'java.util.concurrent.locks.ReentrantReadWriteLock'
-
 module LogStash module Filters module Dictionary
   class DictionaryFileError < StandardError; end
 
   class File
+
+    include LogStash::Util::Loggable
+
     def self.create(path, refresh_interval, refresh_behaviour, exact, regex)
       if /\.y[a]?ml$/.match(path)
         instance = YamlFile.new(path, refresh_interval, exact, regex)
@@ -30,14 +29,12 @@ module LogStash module Filters module Dictionary
       end
     end
 
-    include LogStash::Util::Loggable
     attr_reader :dictionary, :fetch_strategy
 
     def initialize(path, refresh_interval, exact, regex)
       @dictionary_path = path
       @refresh_interval = refresh_interval
       @short_refresh = @refresh_interval <= 300
-      @stopping = Concurrent::AtomicBoolean.new # ported from jdbc_static, need a way to prevent a scheduled execution from running a load.
       rw_lock = java.util.concurrent.locks.ReentrantReadWriteLock.new
       @write_lock = rw_lock.writeLock
       @dictionary = Hash.new
@@ -51,13 +48,6 @@ module LogStash module Filters module Dictionary
               end
       @fetch_strategy = klass.new(*args)
       load_dictionary(raise_exception = true)
-      stop_scheduler(initial = true)
-      start_scheduler unless @refresh_interval <= 0 # disabled, a scheduler interval of zero makes no sense
-    end
-
-    def stop_scheduler(initial = false)
-      @stopping.make_true unless initial
-      @scheduler.shutdown(:wait) if @scheduler
     end
 
     def load_dictionary(raise_exception=false)
@@ -88,13 +78,6 @@ module LogStash module Filters module Dictionary
 
     private
 
-    def start_scheduler
-      @scheduler = Rufus::Scheduler.new
-      @scheduler.interval("#{@refresh_interval}s", :overlap => false) do
-        reload_dictionary
-      end
-    end
-
     def merge_dictionary
       @write_lock.lock
       begin
@@ -116,14 +99,15 @@ module LogStash module Filters module Dictionary
       end
     end
 
+    # scheduler executes this method, periodically
     def reload_dictionary
-      return if @stopping.true?
       if @short_refresh
         load_dictionary if needs_refresh?
       else
         load_dictionary
       end
     end
+    public :reload_dictionary
 
     def needs_refresh?
       @dictionary_mtime != ::File.mtime(@dictionary_path).to_f

data/lib/logstash/filters/translate.rb

@@ -4,6 +4,7 @@ require "logstash/namespace"
 require 'logstash/plugin_mixins/ecs_compatibility_support'
 require 'logstash/plugin_mixins/validator_support/field_reference_validation_adapter'
 require 'logstash/plugin_mixins/deprecation_logger_support'
+require 'logstash/plugin_mixins/scheduler'
 
 require "logstash/filters/dictionary/memory"
 require "logstash/filters/dictionary/file"
@@ -44,6 +45,8 @@ class Translate < LogStash::Filters::Base
 
   extend LogStash::PluginMixins::ValidatorSupport::FieldReferenceValidationAdapter
 
+  include LogStash::PluginMixins::Scheduler
+
   config_name "translate"
 
   # The name of the logstash event field containing the value to be compared for a
@@ -228,11 +231,11 @@ class Translate < LogStash::Filters::Base
     else
       @logger.debug? && @logger.debug("#{self.class.name}: Dictionary translation method - Fuzzy")
     end
-  end # def register
 
-  def close
-    @lookup.stop_scheduler
-  end
+    if @lookup.respond_to?(:reload_dictionary) && @refresh_interval > 0 # a scheduler interval of zero makes no sense
+      scheduler.interval("#{@refresh_interval}s", overlap: false) { @lookup.reload_dictionary }
+    end
+  end # def register
 
   def filter(event)
     return unless @updater.test_for_inclusion(event, @override)

data/logstash-filter-translate.gemspec

@@ -1,7 +1,7 @@
 Gem::Specification.new do |s|
 
   s.name            = 'logstash-filter-translate'
-  s.version         = '3.3.1'
+  s.version         = '3.4.0'
   s.licenses        = ['Apache License (2.0)']
   s.summary         = "Replaces field contents based on a hash or YAML file"
   s.description     = "This gem is a Logstash plugin required to be installed on top of the Logstash core pipeline using $LS_HOME/bin/logstash-plugin install gemname. This gem is not a stand-alone program"
@@ -24,7 +24,7 @@ Gem::Specification.new do |s|
   s.add_runtime_dependency 'logstash-mixin-ecs_compatibility_support', '~> 1.2'
   s.add_runtime_dependency 'logstash-mixin-validator_support', '~> 1.0'
   s.add_runtime_dependency 'logstash-mixin-deprecation_logger_support', '~> 1.0'
-  s.add_runtime_dependency 'rufus-scheduler'
+  s.add_runtime_dependency "logstash-mixin-scheduler", '~> 1.0'
 
   s.add_development_dependency 'logstash-devutils'
   s.add_development_dependency 'rspec-sequencing'

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: logstash-filter-translate
 version: !ruby/object:Gem::Version
-  version: 3.3.1
+  version: 3.4.0
 platform: ruby
 authors:
 - Elastic
@@ -75,17 +75,17 @@ dependencies:
 - !ruby/object:Gem::Dependency
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
-  name: rufus-scheduler
+        version: '1.0'
+  name: logstash-mixin-scheduler
   prerelease: false
   type: :runtime
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '1.0'
 - !ruby/object:Gem::Dependency
   requirement: !ruby/object:Gem::Requirement
     requirements: