Skip to content

Commit

Permalink
Reduce default pipelining to Logstash to a more tame '2'. This reduce…
Browse files Browse the repository at this point in the history
…s memory pressure on LS. (#6250)

We do have improvements to reduce memory usage in LS here logstash-plugins/logstash-input-beats#286

However, I think it makes sense to start with this fix here, and ramp up this number on the next major when we have more confidence that 5 is the magic number.

At the end of the day, magic numbers are always problematic, but 5 can cause quite a bit of pressure when there's a high ratio of Beats:Logstash instances
  • Loading branch information
andrewvc authored and tsg committed Feb 1, 2018
1 parent 67ab782 commit d3a0b94
Show file tree
Hide file tree
Showing 8 changed files with 8 additions and 8 deletions.
2 changes: 1 addition & 1 deletion auditbeat/auditbeat.reference.yml
Original file line number Diff line number Diff line change
Expand Up @@ -356,7 +356,7 @@ output.elasticsearch:

# Number of batches to be sent asynchronously to logstash while processing
# new batches.
#pipelining: 5
#pipelining: 2

# If enabled only a subset of events in a batch of events is transferred per
# transaction. The number of events to be sent increases up to `bulk_max_size`
Expand Down
2 changes: 1 addition & 1 deletion filebeat/filebeat.reference.yml
Original file line number Diff line number Diff line change
Expand Up @@ -804,7 +804,7 @@ output.elasticsearch:

# Number of batches to be sent asynchronously to logstash while processing
# new batches.
#pipelining: 5
#pipelining: 2

# If enabled only a subset of events in a batch of events is transferred per
# transaction. The number of events to be sent increases up to `bulk_max_size`
Expand Down
2 changes: 1 addition & 1 deletion heartbeat/heartbeat.reference.yml
Original file line number Diff line number Diff line change
Expand Up @@ -465,7 +465,7 @@ output.elasticsearch:

# Number of batches to be sent asynchronously to logstash while processing
# new batches.
#pipelining: 5
#pipelining: 2

# If enabled only a subset of events in a batch of events is transferred per
# transaction. The number of events to be sent increases up to `bulk_max_size`
Expand Down
2 changes: 1 addition & 1 deletion libbeat/_meta/config.reference.yml
Original file line number Diff line number Diff line change
Expand Up @@ -251,7 +251,7 @@ output.elasticsearch:

# Number of batches to be sent asynchronously to logstash while processing
# new batches.
#pipelining: 5
#pipelining: 2

# If enabled only a subset of events in a batch of events is transferred per
# transaction. The number of events to be sent increases up to `bulk_max_size`
Expand Down
2 changes: 1 addition & 1 deletion libbeat/outputs/logstash/config.go
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@ type Backoff struct {
var defaultConfig = Config{
Port: 5044,
LoadBalance: false,
Pipelining: 5,
Pipelining: 2,
BulkMaxSize: 2048,
SlowStart: false,
CompressionLevel: 3,
Expand Down
2 changes: 1 addition & 1 deletion metricbeat/metricbeat.reference.yml
Original file line number Diff line number Diff line change
Expand Up @@ -740,7 +740,7 @@ output.elasticsearch:

# Number of batches to be sent asynchronously to logstash while processing
# new batches.
#pipelining: 5
#pipelining: 2

# If enabled only a subset of events in a batch of events is transferred per
# transaction. The number of events to be sent increases up to `bulk_max_size`
Expand Down
2 changes: 1 addition & 1 deletion packetbeat/packetbeat.reference.yml
Original file line number Diff line number Diff line change
Expand Up @@ -719,7 +719,7 @@ output.elasticsearch:

# Number of batches to be sent asynchronously to logstash while processing
# new batches.
#pipelining: 5
#pipelining: 2

# If enabled only a subset of events in a batch of events is transferred per
# transaction. The number of events to be sent increases up to `bulk_max_size`
Expand Down
2 changes: 1 addition & 1 deletion winlogbeat/winlogbeat.reference.yml
Original file line number Diff line number Diff line change
Expand Up @@ -280,7 +280,7 @@ output.elasticsearch:

# Number of batches to be sent asynchronously to logstash while processing
# new batches.
#pipelining: 5
#pipelining: 2

# If enabled only a subset of events in a batch of events is transferred per
# transaction. The number of events to be sent increases up to `bulk_max_size`
Expand Down

0 comments on commit d3a0b94

Please sign in to comment.