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

Reduce logging verbosity for the output filter and Iceberg #20607

Merged
merged 6 commits into from
Oct 2, 2024

Conversation

bernd
Copy link
Member

@bernd bernd commented Oct 1, 2024

Switch log statements in PipelineRuleOutputFilterStateUpdater from "INFO" to "DEBUG.

Adjust log4j2.xml config file to switch verbose Iceberg classes to "warn".

Refs https://github.com/Graylog2/graylog-plugin-enterprise/issues/7703

/nocl

@kodjo-anipah
Copy link
Contributor

@bernd should set the logs in PipelineRuleOutputFilter to debug to?

@bernd
Copy link
Member Author

bernd commented Oct 2, 2024

@bernd should set the logs in PipelineRuleOutputFilter to debug to?

Yes, good catch. Thanks!

Copy link
Contributor

@kodjo-anipah kodjo-anipah left a comment

Choose a reason for hiding this comment

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

LGTM, thanks Bernd!

@bernd bernd merged commit ff7a2aa into master Oct 2, 2024
6 checks passed
@bernd bernd deleted the update/logging-verbosity branch October 2, 2024 11:24
bernd added a commit to Graylog2/fpm-recipes that referenced this pull request Oct 2, 2024
* Sync log4j2.xml file with defaults from the server repo

Refs Graylog2/graylog2-server#20607

* Set log level for org.apache.iceberg.CatalogUtil to warn

* Set log level of org.apache.iceberg.IncrementalFileCleanup to warn
bernd added a commit to Graylog2/graylog-docker that referenced this pull request Oct 2, 2024
* Sync log4j2.xml file with defaults from the server repo

Refs Graylog2/graylog2-server#20607

* Set log level for org.apache.iceberg.CatalogUtil to warn

* Set log level of org.apache.iceberg.IncrementalFileCleanup to warn
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.

2 participants