Skip to content

feat: add max pending entries to all logger plugins#12709

Merged
Revolyssup merged 12 commits into
apache:masterfrom
Revolyssup:revolyssup/max-pend
Oct 30, 2025
Merged

feat: add max pending entries to all logger plugins#12709
Revolyssup merged 12 commits into
apache:masterfrom
Revolyssup:revolyssup/max-pend

Conversation

@Revolyssup

@Revolyssup Revolyssup commented Oct 28, 2025

Copy link
Copy Markdown
Contributor

Batch processor was given a configuration point to set max pending entries which is currently being used in kafka logger. #12338
This PR extends it to other loggers

  • I have explained the need for this PR and the problem it solves
  • I have explained the changes or the new features added to this PR
  • I have added tests corresponding to this change
  • I have updated the documentation to reflect this change
  • I have verified that this change is backward compatible (If not, please discuss on the APISIX mailing list first)

@dosubot dosubot Bot added size:XL This PR changes 500-999 lines, ignoring generated files. enhancement New feature or request labels Oct 28, 2025
@dosubot dosubot Bot added size:L This PR changes 100-499 lines, ignoring generated files. and removed size:XL This PR changes 500-999 lines, ignoring generated files. labels Oct 28, 2025

@membphis membphis left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@Revolyssup
Revolyssup merged commit 2c041a3 into apache:master Oct 30, 2025
46 of 62 checks passed
@Revolyssup
Revolyssup deleted the revolyssup/max-pend branch October 30, 2025 06:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request size:L This PR changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants