Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[libbeat] Enable WriteAheadLimit in the disk queue #21391

Merged
merged 5 commits into from
Sep 29, 2020
Merged

Conversation

faec
Copy link
Contributor

@faec faec commented Sep 29, 2020

This PR activates the previously inert setting WriteAheadLimit, which limits how many events the disk queue will store in memory waiting to be written to disk (this is to prevent unbounded memory use / provide back-pressure when the beat is under extremely high load and is receiving inputs faster than they can be written to disk).

@faec faec added enhancement libbeat needs_backport PR is waiting to be backported to other branches. Team:Integrations Label for the Integrations team labels Sep 29, 2020
@faec faec requested a review from kvch September 29, 2020 15:28
@elasticmachine
Copy link
Collaborator

Pinging @elastic/integrations (Team:Integrations)

@botelastic botelastic bot added needs_team Indicates that the issue/PR needs a Team:* label and removed needs_team Indicates that the issue/PR needs a Team:* label labels Sep 29, 2020
@elasticmachine
Copy link
Collaborator

elasticmachine commented Sep 29, 2020

💚 Build Succeeded

Pipeline View Test View Changes Artifacts preview

Expand to view the summary

Build stats

  • Build Cause: [Pull request #21391 updated]

  • Start Time: 2020-09-29T16:54:50.410+0000

  • Duration: 75 min 9 sec

Test stats 🧪

Test Results
Failed 0
Passed 16178
Skipped 1333
Total 17511

@andresrc
Copy link
Contributor

what is the behaviour when the memory queue is full? blocking, dropping, configurable?

@faec
Copy link
Contributor Author

faec commented Sep 29, 2020

what is the behaviour when the memory queue is full? blocking, dropping, configurable?

Configurable. Blocking vs dropping is unchanged by this PR (in general it is configured by the beat / input producing the events). This PR only changes the conditions under which the queue is considered "full": previously this only meant that it had reached its maximum global byte capacity, whereas now it can also mean that the memory-based intake queue is at capacity.

@faec faec merged commit ef6274d into elastic:master Sep 29, 2020
@faec faec deleted the disk-queue branch September 29, 2020 20:15
faec added a commit to faec/beats that referenced this pull request Sep 29, 2020
@faec faec added v7.10.0 and removed needs_backport PR is waiting to be backported to other branches. labels Sep 29, 2020
v1v added a commit to v1v/beats that referenced this pull request Sep 30, 2020
…ci-build-label-support

* upstream/master:
  [JJBB] Set shallow cloning to 10 (elastic#21409)
  docs: add link to release notes for 7.9.2 (elastic#21405) (elastic#21419)
  docs: Prepare Changelog for 7.9.2 (elastic#21229) (elastic#21403)
  fix: mark flaky tests (elastic#21300)
  fix: use a fixed version of setuptools (elastic#21393)
  Move Kubernetes events metricset to its own block in reference config (elastic#21407)
  [libbeat] Enable WriteAheadLimit in the disk queue (elastic#21391)
  docs: fix apt/yum formatting (elastic#21362)
  Fix shutdown tracking in s3 input (elastic#21380)
  [libbeat] Fix position writing in the disk queue
  Add UBI 8 image to the dependencies report (elastic#21374)
  Fix debug message to show actual SQS message ID (elastic#20614)
  [Elastic Agent] Rename *ConfigChange to PolicyChange (elastic#20779)
  [Elastic Agent] Add install/uninstall sub-command (elastic#21206)
  [Filebeat][httpjson] Make httpjson use cursor input when using date cursor (elastic#20751)
  feat: prepare release pipelines (elastic#21238)
  Add IP validation to Security module (elastic#21325)
faec added a commit that referenced this pull request Sep 30, 2020
v1v added a commit to v1v/beats that referenced this pull request Oct 2, 2020
* upstream/master: (27 commits)
  [Ingest Manager] Split index restrictions into type,dataset, namespace parts (elastic#21406)
  Update Filebeat module expected logs files (elastic#21454)
  Edit SQL module docs and fix broken doc structure (elastic#21233)
  [Ingest Manager] Send snapshot flag together with metadata (elastic#21285)
  Revert "[JJBB] Set shallow cloning to 10 (elastic#21409)" (elastic#21447)
  [JJBB] Use reference repo for fast checkouts (elastic#21410)
  Add initial skeleton of filestream input (elastic#21427)
  Initial spec file for apm-server (elastic#21225)
  [Ingest Manager] Upgrade Action: make source URI optional (elastic#21372)
  Add field limit check for AWS Cloudtrail flattened fields (elastic#21388)
  [Winlogbeat] Move winlogbeat javascript processor to libbeat (elastic#21402)
  ci: pipeline to generate the changelog (elastic#21426)
  [JJBB] Set shallow cloning to 10 (elastic#21409)
  docs: add link to release notes for 7.9.2 (elastic#21405) (elastic#21419)
  docs: Prepare Changelog for 7.9.2 (elastic#21229) (elastic#21403)
  fix: mark flaky tests (elastic#21300)
  fix: use a fixed version of setuptools (elastic#21393)
  Move Kubernetes events metricset to its own block in reference config (elastic#21407)
  [libbeat] Enable WriteAheadLimit in the disk queue (elastic#21391)
  docs: fix apt/yum formatting (elastic#21362)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants