Skip to content

[8.x](backport #42187) [Winlogbeat] Improve query building and error recovery#42210

Merged
marc-gr merged 1 commit into8.xfrom
mergify/bp/8.x/pr-42187
Jan 7, 2025
Merged

[8.x](backport #42187) [Winlogbeat] Improve query building and error recovery#42210
marc-gr merged 1 commit into8.xfrom
mergify/bp/8.x/pr-42187

Conversation

@mergify
Copy link
Copy Markdown
Contributor

@mergify mergify bot commented Jan 3, 2025

Proposed commit message

Improve query building and error recovery:

  • Removes the limitation of 22 clauses by automatically splitting clauses on query building
  • Retries on publisher disabled error in case it is recoverable

Checklist

  • My code follows the style guidelines of this project
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have made corresponding change to the default configuration files
  • I have added tests that prove my fix is effective or that my feature works
  • I have added an entry in CHANGELOG.next.asciidoc or CHANGELOG-developer.next.asciidoc.

Disruptive User Impact

Author's Checklist

  • [ ]

How to test this PR locally

Related issues

Use cases

Screenshots

Logs

This is an automatic backport of pull request #42187 done by [Mergify](https://mergify.com).

@marc-gr @mergify
[Winlogbeat] Improve query building and error recovery (#42187)
88595cc 
* Retry on publisher disabled error

* Split query to never surpass 22 clauses

* Add changelog entry

(cherry picked from commit 279d5cc)
@mergify mergify bot added the backport label Jan 3, 2025
@mergify mergify bot requested a review from a team as a code owner January 3, 2025 16:12
@mergify mergify bot assigned marc-gr Jan 3, 2025
@botelastic botelastic bot added the needs_team Indicates that the issue/PR needs a Team:* label label Jan 3, 2025
@botelastic
Copy link
Copy Markdown

botelastic bot commented Jan 3, 2025

This pull request doesn't have a Team:<team> label.

@mergify
Copy link
Copy Markdown
Contributor Author

mergify bot commented Jan 6, 2025

This pull request has not been merged yet. Could you please review and merge it @marc-gr? 🙏

@marc-gr marc-gr merged commit b8eee43 into 8.x Jan 7, 2025
@marc-gr marc-gr deleted the mergify/bp/8.x/pr-42187 branch January 7, 2025 07:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@marc-gr marc-gr marc-gr approved these changes

Assignees

@marc-gr marc-gr

Labels

backport needs_team Indicates that the issue/PR needs a Team:* label

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@marc-gr