Skip to content

Commit

Permalink
Fix Class Load Order in Netty4Plugin (#42591)
Browse files Browse the repository at this point in the history
* Don't force the logger in the Netty4Plugin class already, at this point log4j might not be fully initialized.
   * The call was redundant anyway since we do the same thing in the Netty4Transport and Netty4HttpServerTransport classes already and there we do it properly after setting up log4j by initilizing the loggers
* Relates #42532
  • Loading branch information
original-brownbear authored May 30, 2019
1 parent 04b7449 commit 67be68d
Showing 1 changed file with 0 additions and 5 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,6 @@
import org.elasticsearch.plugins.Plugin;
import org.elasticsearch.threadpool.ThreadPool;
import org.elasticsearch.transport.netty4.Netty4Transport;
import org.elasticsearch.transport.netty4.Netty4Utils;

import java.util.Arrays;
import java.util.Collections;
Expand All @@ -45,10 +44,6 @@

public class Netty4Plugin extends Plugin implements NetworkPlugin {

static {
Netty4Utils.setup();
}

public static final String NETTY_TRANSPORT_NAME = "netty4";
public static final String NETTY_HTTP_TRANSPORT_NAME = "netty4";

Expand Down

0 comments on commit 67be68d

Please sign in to comment.