You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@trafficcontrol.apache.org by ne...@apache.org on 2017/01/25 17:30:27 UTC

[16/20] incubator-trafficcontrol git commit: Remove TM2 wrong comments

Remove TM2 wrong comments


Project: http://git-wip-us.apache.org/repos/asf/incubator-trafficcontrol/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-trafficcontrol/commit/b2cb7368
Tree: http://git-wip-us.apache.org/repos/asf/incubator-trafficcontrol/tree/b2cb7368
Diff: http://git-wip-us.apache.org/repos/asf/incubator-trafficcontrol/diff/b2cb7368

Branch: refs/heads/master
Commit: b2cb73685c83c30d830b9d2fbc25ea51cd827c5b
Parents: 34863ed
Author: Robert Butts <ro...@gmail.com>
Authored: Mon Jan 23 09:58:23 2017 -0700
Committer: Dave Neuman <ne...@apache.org>
Committed: Wed Jan 25 10:29:46 2017 -0700

----------------------------------------------------------------------
 .../experimental/traffic_monitor/deliveryservice/stat.go           | 1 -
 .../experimental/traffic_monitor/threadsafe/resultstathistory.go   | 2 --
 2 files changed, 3 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-trafficcontrol/blob/b2cb7368/traffic_monitor/experimental/traffic_monitor/deliveryservice/stat.go
----------------------------------------------------------------------
diff --git a/traffic_monitor/experimental/traffic_monitor/deliveryservice/stat.go b/traffic_monitor/experimental/traffic_monitor/deliveryservice/stat.go
index 30e1f74..1958f05 100644
--- a/traffic_monitor/experimental/traffic_monitor/deliveryservice/stat.go
+++ b/traffic_monitor/experimental/traffic_monitor/deliveryservice/stat.go
@@ -39,7 +39,6 @@ import (
 // TODO remove 'ds' and 'stat' from names
 
 // Stats is the JSON-serialisable representation of delivery service Stats. It maps delivery service names to individual stat objects.
-// TODO remove DeliveryService and set type to the map directly, or add other members
 type Stats struct {
 	DeliveryService map[enum.DeliveryServiceName]dsdata.Stat `json:"deliveryService"`
 	Time            time.Time                                `json:"-"`

http://git-wip-us.apache.org/repos/asf/incubator-trafficcontrol/blob/b2cb7368/traffic_monitor/experimental/traffic_monitor/threadsafe/resultstathistory.go
----------------------------------------------------------------------
diff --git a/traffic_monitor/experimental/traffic_monitor/threadsafe/resultstathistory.go b/traffic_monitor/experimental/traffic_monitor/threadsafe/resultstathistory.go
index 966b3ac..cf8f996 100644
--- a/traffic_monitor/experimental/traffic_monitor/threadsafe/resultstathistory.go
+++ b/traffic_monitor/experimental/traffic_monitor/threadsafe/resultstathistory.go
@@ -28,7 +28,6 @@ import (
 
 // ResultStatHistory provides safe access for multiple goroutines readers and a single writer to a stored HistoryHistory object.
 // This could be made lock-free, if the performance was necessary
-// TODO add separate locks for Caches and Deliveryservice maps?
 type ResultStatHistory struct {
 	history *cache.ResultStatHistory
 	m       *sync.RWMutex
@@ -56,7 +55,6 @@ func (h *ResultStatHistory) Set(v cache.ResultStatHistory) {
 
 // ResultStatHistory provides safe access for multiple goroutines readers and a single writer to a stored HistoryHistory object.
 // This could be made lock-free, if the performance was necessary
-// TODO add separate locks for Caches and Deliveryservice maps?
 type ResultInfoHistory struct {
 	history *cache.ResultInfoHistory
 	m       *sync.RWMutex