From a8d622b1b488069e960925f184d6be0c0b51845b Mon Sep 17 00:00:00 2001 From: Andrew Cholakian Date: Thu, 7 Jan 2016 10:55:44 -0600 Subject: [PATCH] Code cleanup per PR review in OutputDelegator Fixes #4391 --- logstash-core/lib/logstash/output_delegator.rb | 14 +++----------- 1 file changed, 3 insertions(+), 11 deletions(-) diff --git a/logstash-core/lib/logstash/output_delegator.rb b/logstash-core/lib/logstash/output_delegator.rb index 051818665..260e6b157 100644 --- a/logstash-core/lib/logstash/output_delegator.rb +++ b/logstash-core/lib/logstash/output_delegator.rb @@ -7,7 +7,7 @@ require "concurrent/atomic/atomic_fixnum" # of 'workers' the output plugin is configured with. # # This plugin also records some basic statistics -module LogStash; class OutputDelegator +module LogStash class OutputDelegator attr_reader :workers, :config, :worker_count, :threadsafe # The *args this takes are the same format that a Outputs::Base takes. A list of hashes with parameters in them @@ -145,14 +145,6 @@ module LogStash; class OutputDelegator end private - - # Needed for tests - def threadsafe_worker - @threadsafe_worker - end - - # Needed for tests - def worker_queue - @worker_queue - end + # Needed for testing, so private + attr_reader :threadsafe_worker, :worker_queue end end \ No newline at end of file