Skip to content

Commit

Permalink
[ML] Fix logger declaration in ML plugins (#42222)
Browse files Browse the repository at this point in the history
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`.
  • Loading branch information
dimitris-athanasiou committed May 21, 2019
1 parent de09648 commit 3019092
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 8 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -76,10 +76,8 @@
import java.util.Arrays;
import java.util.Collection;
import java.util.Collections;
import java.util.HashSet;
import java.util.List;
import java.util.Map;
import java.util.Set;
import java.util.function.Supplier;
import java.util.function.UnaryOperator;

Expand All @@ -90,11 +88,7 @@ public class DataFrame extends Plugin implements ActionPlugin, PersistentTaskPlu
public static final String NAME = "data_frame";
public static final String TASK_THREAD_POOL_NAME = "data_frame_indexing";

// list of headers that will be stored when a transform is created
public static final Set<String> HEADER_FILTERS = new HashSet<>(
Arrays.asList("es-security-runas-user", "_xpack_security_authentication"));

private static final Logger logger = LogManager.getLogger(XPackPlugin.class);
private static final Logger logger = LogManager.getLogger(DataFrame.class);

private final boolean enabled;
private final Settings settings;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -300,7 +300,7 @@ public class MachineLearning extends Plugin implements ActionPlugin, AnalysisPlu
public static final Setting<ByteSizeValue> MIN_DISK_SPACE_OFF_HEAP =
Setting.byteSizeSetting("xpack.ml.min_disk_space_off_heap", new ByteSizeValue(5, ByteSizeUnit.GB), Setting.Property.NodeScope);

private static final Logger logger = LogManager.getLogger(XPackPlugin.class);
private static final Logger logger = LogManager.getLogger(MachineLearning.class);

private final Settings settings;
private final Environment env;
Expand Down

0 comments on commit 3019092

Please sign in to comment.