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

Document the limitation of log message indexing #18069

Merged

Conversation

loicmathieu
Copy link
Contributor

Fixes #15370

Copy link
Member

@gsmet gsmet left a comment

Choose a reason for hiding this comment

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

Nice thanks! I added a small suggestion.

docs/src/main/asciidoc/centralized-log-management.adoc Outdated Show resolved Hide resolved
Fixes quarkusio#15370
WIP Update docs/src/main/asciidoc/centralized-log-management.adoc
@loicmathieu loicmathieu force-pushed the doc/log-message-indexing-limitation branch from d92c4ac to 3e89e28 Compare June 22, 2021 12:27
@gsmet gsmet added the triage/waiting-for-ci Ready to merge when CI successfully finishes label Jun 22, 2021
@loicmathieu
Copy link
Contributor Author

@gsmet thanks for the review!
This verb should be added to the english dictionary, we do so many workarounds in software programming ;)

@gsmet gsmet merged commit d25474b into quarkusio:main Jun 22, 2021
@quarkus-bot quarkus-bot bot added this to the 2.1 - main milestone Jun 22, 2021
@quarkus-bot quarkus-bot bot removed the triage/waiting-for-ci Ready to merge when CI successfully finishes label Jun 22, 2021
@gsmet gsmet modified the milestones: 2.1 - main, 2.0.0.Final Jun 22, 2021
@loicmathieu loicmathieu deleted the doc/log-message-indexing-limitation branch June 22, 2021 16:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

quarkus-logging-gelf: mapper_parsing_exception when include-log-message-parameters is true (default)
2 participants