Skip to content
This repository has been archived by the owner on Jul 21, 2021. It is now read-only.

ServerStats should hold time.Durations for latencies, not int64 #116

Open
langston-barrett opened this issue Apr 13, 2016 · 0 comments
Open

Comments

@langston-barrett
Copy link

Currently, ServerStats holds

type ServerStats struct {
    Sent        int64
    Received    int64
    NodeCount   int64
    MinLatency  int64
    AvgLatency  int64
    MaxLatency  int64
    Connections int64
    Outstanding int64
    Epoch       int32
    Counter     int32
    BuildTime   time.Time
    Mode        Mode
    Version     string
    Error       error
}

Whereas this should probably be

type ServerStats struct {
    Sent        int64
    Received    int64
    NodeCount   int64
    MinLatency  time.Duration
    AvgLatency  time.Duration
    MaxLatency  time.Duration
    Connections int64
    Outstanding int64
    Epoch       int32
    Counter     int32
    BuildTime   time.Time
    Mode        Mode
    Version     string
    Error       error
}
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant