github-mirrors/logstash
1
0
Fork 0
mirror of https://github.com/elastic/logstash.git synced 2025-04-24 14:47:19 -04:00
Code Issues Projects Releases Packages Wiki Activity

MINOR: Remove needless lock on acked batch read

Browse source 
* The underlying call has its own lock, so this extra mutex is not necessary

Fixes #8338
This commit is contained in:
Armin 2017-09-20 16:26:41 +02:00 • committed by Armin Braun
parent e1f97e5d83
commit aca01544a4
1 changed files with 1 additions and 6 deletions
Download patch file Download diff file Expand all files Collapse all files

7
logstash-core/lib/logstash/util/wrapped_acked_queue.rb
View file

@ -167,12 +167,7 @@ module LogStash; module Util
end
batch = new_batch
@mutex.lock
begin
batch.read_next
ensure
@mutex.unlock
end
batch.read_next
start_metrics(batch)
batch
end