[UNDERTOW-2474]-Create a default constant for UndertowOptions.ENABLE_STATISTICS

Author: mugdha273

core/src/main/java/io/undertow/UndertowOptions.java

@@ -238,12 +238,18 @@ public class UndertowOptions {
      */
     public static final Option<Boolean> ENABLE_HTTP2 = Option.simple(UndertowOptions.class, "ENABLE_HTTP2", Boolean.class);
 
+    /**
+     * Default value of {@link #ENABLE_STATISTICS} option.
+     */
+    public static final boolean DEFAULT_ENABLE_STATISTICS = false;
+
     /**
      * If connector level statistics should be enabled. This has a slight performance impact, but allows statistics such
      * as bytes sent/recevied to be monitored.
      *
      * If this is passed to the client then client statistics will be enabled.
-     *
+     * <p>
+     * Defaults to {@link #DEFAULT_ENABLE_STATISTICS}
      */
     public static final Option<Boolean> ENABLE_STATISTICS = Option.simple(UndertowOptions.class, "ENABLE_STATISTICS", Boolean.class);
 

core/src/main/java/io/undertow/client/ajp/AjpClientProvider.java

@@ -113,7 +113,7 @@ private void handleConnected(StreamConnection connection, ClientCallback<ClientC
 
         final ClientStatisticsImpl clientStatistics;
         //first we set up statistics, if required
-        if (options.get(UndertowOptions.ENABLE_STATISTICS, false)) {
+        if (options.get(UndertowOptions.ENABLE_STATISTICS, UndertowOptions.DEFAULT_ENABLE_STATISTICS)) {
             clientStatistics = new ClientStatisticsImpl();
             connection.getSinkChannel().setConduit(new BytesSentStreamSinkConduit(connection.getSinkChannel().getConduit(), new ByteActivityCallback() {
                 @Override

core/src/main/java/io/undertow/client/http/HttpClientConnection.java

@@ -150,7 +150,7 @@ public void handleEvent(StreamSourceConduit channel) {
     HttpClientConnection(final StreamConnection connection, final OptionMap options, final ByteBufferPool bufferPool) {
 
         //first we set up statistics, if required
-        if(options.get(UndertowOptions.ENABLE_STATISTICS, false)) {
+        if(options.get(UndertowOptions.ENABLE_STATISTICS, UndertowOptions.DEFAULT_ENABLE_STATISTICS)) {
             clientStatistics = new ClientStatisticsImpl();
             connection.getSinkChannel().setConduit(new BytesSentStreamSinkConduit(connection.getSinkChannel().getConduit(), new ByteActivityCallback() {
                 @Override

core/src/main/java/io/undertow/client/http2/Http2ClearClientProvider.java

@@ -207,7 +207,7 @@ public void handleEvent(StreamConnection channel) {
 
             final ClientStatisticsImpl clientStatistics;
             //first we set up statistics, if required
-            if (options.get(UndertowOptions.ENABLE_STATISTICS, false)) {
+            if (options.get(UndertowOptions.ENABLE_STATISTICS, UndertowOptions.DEFAULT_ENABLE_STATISTICS)) {
                 clientStatistics = new ClientStatisticsImpl();
                 channel.getSinkChannel().setConduit(new BytesSentStreamSinkConduit(channel.getSinkChannel().getConduit(), new ByteActivityCallback() {
                     @Override

core/src/main/java/io/undertow/client/http2/Http2ClientProvider.java

@@ -155,7 +155,7 @@ private static Http2ClientConnection createHttp2Channel(StreamConnection connect
 
         final ClientStatisticsImpl clientStatistics;
         //first we set up statistics, if required
-        if (options.get(UndertowOptions.ENABLE_STATISTICS, false)) {
+        if (options.get(UndertowOptions.ENABLE_STATISTICS, UndertowOptions.DEFAULT_ENABLE_STATISTICS)) {
             clientStatistics = new ClientStatisticsImpl();
             connection.getSinkChannel().setConduit(new BytesSentStreamSinkConduit(connection.getSinkChannel().getConduit(), new ByteActivityCallback() {
                 @Override

core/src/main/java/io/undertow/client/http2/Http2PriorKnowledgeClientProvider.java

@@ -113,7 +113,7 @@ private void handleConnected(final StreamConnection connection, final ClientCall
 
             final ClientStatisticsImpl clientStatistics;
             //first we set up statistics, if required
-            if (options.get(UndertowOptions.ENABLE_STATISTICS, false)) {
+            if (options.get(UndertowOptions.ENABLE_STATISTICS, UndertowOptions.DEFAULT_ENABLE_STATISTICS)) {
                 clientStatistics = new ClientStatisticsImpl();
                 connection.getSinkChannel().setConduit(new BytesSentStreamSinkConduit(connection.getSinkChannel().getConduit(), new ByteActivityCallback() {
                     @Override

core/src/main/java/io/undertow/server/protocol/http2/Http2OpenListener.java

@@ -109,7 +109,7 @@ public Http2OpenListener(final ByteBufferPool pool, final OptionMap undertowOpti
         this.bufferSize = buf.getBuffer().remaining();
         buf.close();
         connectorStatistics = new ConnectorStatisticsImpl();
-        statisticsEnabled = undertowOptions.get(UndertowOptions.ENABLE_STATISTICS, false);
+        statisticsEnabled = undertowOptions.get(UndertowOptions.ENABLE_STATISTICS, UndertowOptions.DEFAULT_ENABLE_STATISTICS);
         this.protocol = protocol;
     }