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

Fix error log class name issue #75831

Merged
merged 2 commits into from
Aug 4, 2021
Merged

Conversation

howardhuanghua
Copy link
Contributor

Fix error log name issue.

@howardhuanghua howardhuanghua changed the title Fix error log name issue. Fix error log class name issue. Jul 29, 2021
@elasticsearchmachine elasticsearchmachine added v8.0.0 external-contributor Pull request authored by a developer outside the Elasticsearch team labels Jul 29, 2021
@mark-vieira mark-vieira added the :Core/Infra/Logging Log management and logging utilities label Jul 29, 2021
@elasticmachine elasticmachine added the Team:Core/Infra Meta label for core/infra team label Jul 29, 2021
@elasticmachine
Copy link
Collaborator

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

@gwbrown gwbrown self-requested a review July 29, 2021 20:14
@gwbrown
Copy link
Contributor

gwbrown commented Jul 29, 2021

@elasticmachine test this please

@gwbrown
Copy link
Contributor

gwbrown commented Jul 29, 2021

@elasticmachine update branch

@elasticmachine
Copy link
Collaborator

user doesn't have permission to update head repository

@gwbrown
Copy link
Contributor

gwbrown commented Jul 29, 2021

@howardhuanghua Could you please merge master into your branch? It doesn't appear we have permission to do that automatically.

@howardhuanghua
Copy link
Contributor Author

@gwbrown Thanks, branch has been updated.

@gwbrown
Copy link
Contributor

gwbrown commented Jul 30, 2021

@elasticmachine test this please

@gwbrown gwbrown self-assigned this Jul 30, 2021
@howardhuanghua
Copy link
Contributor Author

@gwbrown Would you please help to merge this simple PR? It seems all checks have hassed.

@pugnascotia pugnascotia changed the title Fix error log class name issue. Fix error log class name issue Aug 4, 2021
@pugnascotia pugnascotia merged commit 63ada76 into elastic:master Aug 4, 2021
@elasticsearchmachine
Copy link
Collaborator

💔 Backport failed

The backport operation could not be completed due to the following error:
There are no branches to backport to. Aborting.

To backport manually run:
backport --pr 75831

pugnascotia pushed a commit that referenced this pull request Aug 4, 2021
AbstractIndexOrdinalsFieldData referenced the wrong class in its logger. Fix it.
@pugnascotia
Copy link
Contributor

Backported to 7.x in f0603fa.

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 external-contributor Pull request authored by a developer outside the Elasticsearch team >non-issue Team:Core/Infra Meta label for core/infra team v7.15.0 v8.0.0-alpha2
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants