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

Add new filter to disable query logging: ep_disable_query_logging #4019

Open
wants to merge 5 commits into
base: develop
Choose a base branch
from

Conversation

rebeccahum
Copy link
Contributor

Description of the Change

Adds a new filter called ep_disable_query_logging to disable query logging due to potential memory leaks.

Closes #4006

How to test the Change

add_filter( 'ep_disable_query_logging', '__return_true' );

Changelog Entry

Added - New filter ep_disable_query_logging to disable query logging

Credits

Props @davidsword

Checklist:

@rebeccahum rebeccahum force-pushed the add/filter-ep_disable_query_logging branch from ef9f723 to e2e0571 Compare November 26, 2024 20:10
@felipeelia
Copy link
Member

Thanks @rebeccahum. Any chance you can add a docblock to that new filter following the pattern we have for the other filters?

@rebeccahum rebeccahum force-pushed the add/filter-ep_disable_query_logging branch 2 times, most recently from 7426929 to 0456bec Compare November 26, 2024 20:15
@rebeccahum rebeccahum force-pushed the add/filter-ep_disable_query_logging branch from 0456bec to 6f94523 Compare November 26, 2024 20:35
@rebeccahum rebeccahum force-pushed the add/filter-ep_disable_query_logging branch 2 times, most recently from 4d54c46 to 84541d9 Compare November 26, 2024 22:01
@rebeccahum rebeccahum force-pushed the add/filter-ep_disable_query_logging branch from 84541d9 to f0581f8 Compare November 26, 2024 22:13
@rebeccahum
Copy link
Contributor Author

@felipeelia Done! Thank you

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.

BUG: query logging memory hog
2 participants