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

[libbeat] Remove global loggers from jsontransform #18550

Merged
merged 1 commit into from
May 19, 2020
Merged

[libbeat] Remove global loggers from jsontransform #18550

merged 1 commit into from
May 19, 2020

Conversation

kaiyan-sheng
Copy link
Contributor

@kaiyan-sheng kaiyan-sheng commented May 14, 2020

This PR is to remove global loggers for libbeat/common/jsontransform.
Please see #15699 for the meta-issue and more details.

@botelastic botelastic bot added the needs_team Indicates that the issue/PR needs a Team:* label label May 14, 2020
@kaiyan-sheng kaiyan-sheng self-assigned this May 14, 2020
@kaiyan-sheng kaiyan-sheng added needs_backport PR is waiting to be backported to other branches. refactoring review Team:Integrations Label for the Integrations team and removed needs_team Indicates that the issue/PR needs a Team:* label labels May 14, 2020
@elasticmachine
Copy link
Collaborator

Pinging @elastic/integrations (Team:Integrations)

@elasticmachine
Copy link
Collaborator

elasticmachine commented May 14, 2020

💚 Build Succeeded

Pipeline View Test View Changes Artifacts preview

Expand to view the summary

Build stats

Test stats 🧪

Test Results
Failed 0
Passed 7772
Skipped 1236
Total 9008

@kaiyan-sheng kaiyan-sheng merged commit f54c6f5 into elastic:master May 19, 2020
@kaiyan-sheng kaiyan-sheng deleted the logger_json_helper branch May 19, 2020 18:11
@kaiyan-sheng kaiyan-sheng added v7.9.0 and removed needs_backport PR is waiting to be backported to other branches. labels May 19, 2020
kaiyan-sheng added a commit that referenced this pull request May 20, 2020
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.

4 participants