Skip to content

Commit

Permalink
log time in millis instead of seconds.nanos
Browse files Browse the repository at this point in the history
  • Loading branch information
gpolitis committed May 29, 2019
1 parent 0de7565 commit 5cbe5f8
Showing 1 changed file with 2 additions and 18 deletions.
20 changes: 2 additions & 18 deletions src/main/java/org/jitsi/utils/logging/DiagnosticContext.java
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,6 @@
*/
package org.jitsi.utils.logging;

import java.time.*;
import java.util.*;
import java.util.concurrent.*;

Expand All @@ -37,7 +36,7 @@ public class DiagnosticContext
*/
public TimeSeriesPoint makeTimeSeriesPoint(String timeSeriesName)
{
return makeTimeSeriesPoint(timeSeriesName, Instant.now());
return makeTimeSeriesPoint(timeSeriesName, System.currentTimeMillis());
}

/**
Expand All @@ -48,25 +47,10 @@ public TimeSeriesPoint makeTimeSeriesPoint(String timeSeriesName)
* @param tsMs the timestamp of the time series point (in millis)
*/
public TimeSeriesPoint makeTimeSeriesPoint(String timeSeriesName, long tsMs)
{
return makeTimeSeriesPoint(
timeSeriesName, Instant.ofEpochMilli(tsMs));
}

/**
* Makes a new time series point with a timestamp. This is recommended for
* time series where it's important to have the exact timestamp value.
*
* @param timeSeriesName the name of the time series
* @param instant the timestamp of the time series point (in millis)
*/
public TimeSeriesPoint makeTimeSeriesPoint(
String timeSeriesName, Instant instant)
{
return new TimeSeriesPoint(this)
.addField("series", timeSeriesName)
.addField("time",
instant.getEpochSecond() + "." + instant.getNano());
.addField("time", tsMs);
}

public static class TimeSeriesPoint
Expand Down

0 comments on commit 5cbe5f8

Please sign in to comment.