// Copyright (C) 2015 The Android Open Source Project
//
// 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 com.google.gerrit.server.git;

import static java.util.concurrent.TimeUnit.MILLISECONDS;

import com.google.gerrit.metrics.Counter1;
import com.google.gerrit.metrics.Description;
import com.google.gerrit.metrics.Description.Units;
import com.google.gerrit.metrics.Field;
import com.google.gerrit.metrics.Histogram1;
import com.google.gerrit.metrics.MetricMaker;
import com.google.gerrit.metrics.Timer1;
import com.google.inject.Inject;
import com.google.inject.Singleton;

import org.eclipse.jgit.storage.pack.PackStatistics;
import org.eclipse.jgit.transport.PostUploadHook;

@Singleton
public class UploadPackMetricsHook implements PostUploadHook {
  enum Operation {
    CLONE,
    FETCH;
  }

  private final Counter1<Operation> requestCount;
  private final Timer1<Operation> counting;
  private final Timer1<Operation> compressing;
  private final Timer1<Operation> writing;
  private final Histogram1<Operation> packBytes;

  @Inject
  UploadPackMetricsHook(MetricMaker metricMaker) {
    Field<Operation> operation = Field.ofEnum(Operation.class, "operation");
    requestCount = metricMaker.newCounter(
        "git/upload-pack/request_count",
        new Description("Total number of git-upload-pack requests")
          .setRate()
          .setUnit("requests"),
        operation);

    counting = metricMaker.newTimer(
        "git/upload-pack/phase_counting",
        new Description("Time spenting in the 'Counting...' phase")
          .setCumulative()
          .setUnit(Units.MILLISECONDS),
        operation);

    compressing = metricMaker.newTimer(
        "git/upload-pack/phase_compressing",
        new Description("Time spenting in the 'Compressing...' phase")
          .setCumulative()
          .setUnit(Units.MILLISECONDS),
        operation);

    writing = metricMaker.newTimer(
        "git/upload-pack/phase_writing",
        new Description("Time spenting transferring bytes to client")
          .setCumulative()
          .setUnit(Units.MILLISECONDS),
        operation);

    packBytes = metricMaker.newHistogram(
        "git/upload-pack/pack_bytes",
        new Description("Distribution of sizes of packs sent to clients")
          .setCumulative()
          .setUnit(Units.BYTES),
        operation);
  }

  @Override
  public void onPostUpload(PackStatistics stats) {
    Operation op = Operation.FETCH;
    if (stats.getUninterestingObjects() == null
        || stats.getUninterestingObjects().isEmpty()) {
      op = Operation.CLONE;
    }

    requestCount.increment(op);
    counting.record(op, stats.getTimeCounting(), MILLISECONDS);
    compressing.record(op, stats.getTimeCompressing(), MILLISECONDS);
    writing.record(op, stats.getTimeWriting(), MILLISECONDS);
    packBytes.record(op, stats.getTotalBytes());
  }
}

@@ -2011,35 +2011,32 @@
 tion(%22Time spent
-ing
  in the 'Countin
@@ -2041,32 +2041,32 @@
 ing...' phase%22)%0A
+
           .setCu
@@ -2257,19 +2257,16 @@
 me spent
-ing
  in the 
@@ -2451,24 +2451,24 @@
 e_writing%22,%0A
+
         new 
@@ -2490,19 +2490,16 @@
 me spent
-ing
  transfe

Fix spelling errors