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

Turn off transformed statement logging by default #476

Open
bmtcril opened this issue Dec 13, 2024 · 0 comments
Open

Turn off transformed statement logging by default #476

bmtcril opened this issue Dec 13, 2024 · 0 comments
Labels
good first issue A good task for a newcomer to start with

Comments

@bmtcril
Copy link
Contributor

bmtcril commented Dec 13, 2024

Right now we log everything by default, which can be spammy and a waste of resources if operators aren't storing that data or using something like Vector in Aspects to read it. These seem to be relatively rare use cases, so we should change them to be off by default.

XAPI_EVENT_LOGGING_ENABLED = SettingToggle("XAPI_EVENT_LOGGING_ENABLED", default=True)

CALIPER_EVENT_LOGGING_ENABLED = SettingToggle("CALIPER_EVENT_LOGGING_ENABLED", default=True)

@bmtcril bmtcril moved this to Ready for Work in Data Working Group Dec 13, 2024
@saraburns1 saraburns1 added the good first issue A good task for a newcomer to start with label Jan 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue A good task for a newcomer to start with
Projects
Status: Ready for Work
Development

No branches or pull requests

2 participants