Skip to content

Conversation

@pgomulka
Copy link
Contributor

@pgomulka pgomulka commented Sep 24, 2020

known issue doc on a bug #60561

@pgomulka pgomulka added >docs General docs changes :Core/Infra/Logging Log management and logging utilities v6.8.13 labels Sep 24, 2020
@pgomulka pgomulka self-assigned this Sep 24, 2020
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-core-infra (:Core/Infra/Logging)

@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-docs (>docs)

@elasticmachine elasticmachine added Team:Core/Infra Meta label for core/infra team Team:Docs Meta label for docs team labels Sep 24, 2020
Copy link
Contributor

@jrodewig jrodewig left a comment

Choose a reason for hiding this comment

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

Looks good overall. I left a comment regarding clarity.

Comment on lines 210 to 214
Deprecation Logging::

In a mixed cluster environment where one of the nodes is 6.7 but coordinating node is 6.7
an exception will be thrown when a deprecation warning is emitted on pre 6.7 node.
it was introduced by (PR: {es-pull}/37622[#37622]) in order to speed up the performance of header warning parsing.
Copy link
Contributor

@jrodewig jrodewig Sep 28, 2020

Choose a reason for hiding this comment

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

It's not clear which node is pre-6.7.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

good point, it did not made sense :D thanks

@pgomulka pgomulka merged commit 6a92ef5 into elastic:6.8 Sep 29, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Core/Infra/Logging Log management and logging utilities >docs General docs changes Team:Core/Infra Meta label for core/infra team Team:Docs Meta label for docs team v6.8.13

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants