-
Notifications
You must be signed in to change notification settings - Fork 24.9k
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
[ML] Fix logger declaration in ML plugins #42222
[ML] Fix logger declaration in ML plugins #42222
Conversation
This corrects what appears to have been a copy-paste error where the logger for `MachineLearning` and `DataFrame` was wrongly set to be that of `XPackPlugin`.
Pinging @elastic/ml-core |
@@ -94,7 +94,7 @@ | |||
public static final Set<String> HEADER_FILTERS = new HashSet<>( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@hendrikmuhs Also, this appears unused. Are there plans of using it?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it's been refactored out into ClientHelper.SECURITY_HEADER_FILTERS
(if you search for usages of that you'll see it's used in the dataframe
plugin), but the old duplicated variable has not been removed in this file.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
OK, I'll remove it then in this PR.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@@ -94,7 +94,7 @@ | |||
public static final Set<String> HEADER_FILTERS = new HashSet<>( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it's been refactored out into ClientHelper.SECURITY_HEADER_FILTERS
(if you search for usages of that you'll see it's used in the dataframe
plugin), but the old duplicated variable has not been removed in this file.
This corrects what appears to have been a copy-paste error where the logger for `MachineLearning` and `DataFrame` was wrongly set to be that of `XPackPlugin`.
This corrects what appears to have been a copy-paste error where the logger for `MachineLearning` and `DataFrame` was wrongly set to be that of `XPackPlugin`.
This corrects what appears to have been a copy-paste error
where the logger for
MachineLearning
andDataFrame
was wronglyset to be that of
XPackPlugin
.