/*
 * Copyright 2017 LinkedIn Corp.
 *
 * Licensed under the Apache License, Version 2.0 (the "License"); you may not
 * use this file except in compliance with the License. You may obtain a copy of
 * the License at
 *
 * http://www.apache.org/licenses/LICENSE-2.0
 *
 * Unless required by applicable law or agreed to in writing, software
 * distributed under the License is distributed on an "AS IS" BASIS, WITHOUT
 * WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the
 * License for the specific language governing permissions and limitations under
 * the License.
 */

package azkaban.webapp;

import azkaban.executor.ExecutorManager;
import azkaban.metrics.MetricsManager;
import azkaban.metrics.MetricsUtility;

import com.codahale.metrics.Meter;
import com.codahale.metrics.MetricRegistry;

import java.util.concurrent.atomic.AtomicLong;
import java.util.function.Supplier;


/**
 * This singleton class WebMetrics is in charge of collecting varieties of metrics
 * from azkaban-web-server modules.
 */
public enum WebMetrics {
  INSTANCE;

  private MetricRegistry registry;

  private Meter webGetCall;
  private Meter webPostCall;

  // How long does user log fetch take when user call fetch-log api.
  private AtomicLong logFetchLatency = new AtomicLong(0L);

  WebMetrics() {
    registry = MetricsManager.INSTANCE.getRegistry();
    setupStaticMetrics();
  }

  private void setupStaticMetrics() {
    webGetCall = MetricsUtility.addMeter("Web-Get-Call-Meter", registry);
    webPostCall = MetricsUtility.addMeter("Web-Post-Call-Meter", registry);
    MetricsUtility.addGauge("fetchLogLatency", registry, logFetchLatency::get);
  }

  public void markWebGetCall() {

    /*
     * This method should be Thread Safe.
     * Two reasons that we don't make this function call synchronized:
     * 1). drop wizard metrics deals with concurrency internally;
     * 2). mark is basically a math addition operation, which should not cause race condition issue.
     */
    webGetCall.mark();
  }

  public void markWebPostCall() {

    webPostCall.mark();
  }

  public void setFetchLogLatency(long milliseconds) {
    logFetchLatency.set(milliseconds);
  }
}

@@ -617,49 +617,8 @@
 p;%0A%0A
-import azkaban.executor.ExecutorManager;%0A
 impo
@@ -823,44 +823,8 @@
 ng;%0A
-import java.util.function.Supplier;%0A
 %0A%0A/*
@@ -996,16 +996,22 @@
 private 
+final 
 MetricRe
@@ -1030,32 +1030,38 @@
 try;%0A%0A  private 
+final 
 Meter webGetCall
@@ -1072,16 +1072,22 @@
 private 
+final 
 Meter we
@@ -1177,16 +1177,22 @@
 private 
+final 
 AtomicLo
@@ -1308,77 +1308,8 @@
 ();%0A
-    setupStaticMetrics();%0A  %7D%0A%0A  private void setupStaticMetrics() %7B%0A
     

Fix build warning : enums should be immutable. (#1031)