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

Suppress long mapping logging during mapping updates (unless in TRACE) #7949

Closed
wants to merge 1 commit into from

Conversation

bleskes
Copy link
Contributor

@bleskes bleskes commented Oct 1, 2014

Currently DEBUG logs can get very verbose because IndicesClusterStateService logs the complete mapping with every mapping update. We should suppress it if long in DEBUG mode and always log the full one in TRACE.

… in TRACE)

Currently DEBUG logs can get very verbose because IndicesClusterStateService logs the complete mapping with every mapping update. We should suppress it if long in DEBUG mode and always log the full one in TRACE.
@s1monw
Copy link
Contributor

s1monw commented Oct 2, 2014

LGTM

@s1monw s1monw removed the review label Oct 2, 2014
@bleskes bleskes closed this in 1cc5da4 Oct 2, 2014
bleskes added a commit that referenced this pull request Oct 2, 2014
… in TRACE)

Currently DEBUG logs can get very verbose because IndicesClusterStateService logs the complete mapping with every mapping update. We should suppress it if long in DEBUG mode and always log the full one in TRACE.

Closes #7949
@bleskes bleskes deleted the mapping_logging branch October 2, 2014 20:23
@clintongormley clintongormley added the :Core/Infra/Logging Log management and logging utilities label Mar 19, 2015
@clintongormley clintongormley changed the title Logging: suppress long mapping logging during mapping updates (unless in TRACE) Suppress long mapping logging during mapping updates (unless in TRACE) Jun 7, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants