Skip to content

feat: support configuring the buffer size of the access log#10225

Merged
juststillthinking merged 1 commit into
apache:masterfrom
qihaiyan:fix-10219
Sep 19, 2023
Merged

feat: support configuring the buffer size of the access log#10225
juststillthinking merged 1 commit into
apache:masterfrom
qihaiyan:fix-10219

Conversation

@qihaiyan

@qihaiyan qihaiyan commented Sep 18, 2023

Copy link
Copy Markdown
Contributor

Description

access log's buffer size can be configed.

Access log buffering can significantly reduce disk I/O and improve server performance. Enabling access log buffering is relatively simple in NGINX. To do this, use the following directive in your NGINX configuration:

access_log /var/log/nginx/access.log main buffer=32k;
This directive tells NGINX to store the access log entries in a buffer of size 32k before writing them to the log file. Increasing the buffer size can minimize the number of write operations to the disk, resulting in improved performance.

Fixes #10219

Checklist

  • 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)

@qihaiyan qihaiyan changed the title access log's buffer size can be configed feat: access log's buffer size can be configed Sep 18, 2023
@shreemaan-abhishek

Copy link
Copy Markdown
Contributor

Please also describe why this is needed.

@qihaiyan

Copy link
Copy Markdown
Contributor Author

Please also describe why this is needed.

I enriched the description.

@juststillthinking juststillthinking changed the title feat: access log's buffer size can be configed feat: support configuring the buffer size of the access log Sep 19, 2023
@juststillthinking
juststillthinking merged commit 57a7dcd into apache:master Sep 19, 2023
@qihaiyan
qihaiyan deleted the fix-10219 branch September 20, 2023 01:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

feat: As a user, I want to access log's buffer size can be configed in config file

3 participants