logstash-plugins/logstash-output-elasticsearch

Default values of `http_compression` and `compression_level` are incosistent

andsel opened this issue · 2 comments

Tell us about the issue

Description:

With version 11.17.0 the setting http_compression (default value false) was deprecated and replaced by compression_level (default value 1, which is the minimal compression level).
Given that compression_level by default is 1, which means "compression enabled" is opposite to the (now deprecated) default value of http_compression .

Update the default value to http_compression tu true.

Maybe it's not just documentation change.

URL:

Example: https://www.elastic.co/guide/en/logstash/current/introduction.html

Anything else?

Yes, http compression is on by default, but the http_compression config option itself effectively no longer has a default value because it is now simply a deprecated alias that is only consulted when (a) it is explicitly given and (b) the preferred compression_level config option is not explicitly given.

I would close this with "works as expected no changes requested" motivation