[GitHub] [trafficcontrol] ocket8888 commented on a change in pull request #3758: Rewrote deliveryservice_stats to Go

2019-10-01 Thread GitBox
ocket commented on a change in pull request #3758: Rewrote deliveryservice_stats to Go URL: https://github.com/apache/trafficcontrol/pull/3758#discussion_r330308059 ## File path: docs/source/api/deliveryservice_stats.rst ## @@ -37,140 +33,184 @@ Request Structure

[GitHub] [trafficcontrol] ocket8888 commented on a change in pull request #3758: Rewrote deliveryservice_stats to Go

2019-09-30 Thread GitBox
ocket commented on a change in pull request #3758: Rewrote deliveryservice_stats to Go URL: https://github.com/apache/trafficcontrol/pull/3758#discussion_r329785455 ## File path: traffic_ops/traffic_ops_golang/trafficstats/trafficstats.go ## @@ -0,0 +1,691 @@ +package

[GitHub] [trafficcontrol] ocket8888 commented on a change in pull request #3758: Rewrote deliveryservice_stats to Go

2019-09-30 Thread GitBox
ocket commented on a change in pull request #3758: Rewrote deliveryservice_stats to Go URL: https://github.com/apache/trafficcontrol/pull/3758#discussion_r329785455 ## File path: traffic_ops/traffic_ops_golang/trafficstats/trafficstats.go ## @@ -0,0 +1,691 @@ +package

[GitHub] [trafficcontrol] ocket8888 commented on a change in pull request #3758: Rewrote deliveryservice_stats to Go

2019-09-30 Thread GitBox
ocket commented on a change in pull request #3758: Rewrote deliveryservice_stats to Go URL: https://github.com/apache/trafficcontrol/pull/3758#discussion_r329780962 ## File path: traffic_ops/traffic_ops_golang/trafficstats/trafficstats.go ## @@ -0,0 +1,691 @@ +package

[GitHub] [trafficcontrol] ocket8888 commented on a change in pull request #3758: Rewrote deliveryservice_stats to Go

2019-09-27 Thread GitBox
ocket commented on a change in pull request #3758: Rewrote deliveryservice_stats to Go URL: https://github.com/apache/trafficcontrol/pull/3758#discussion_r329239373 ## File path: traffic_ops/traffic_ops_golang/trafficstats/trafficstats.go ## @@ -0,0 +1,691 @@ +package

[GitHub] [trafficcontrol] ocket8888 commented on a change in pull request #3758: Rewrote deliveryservice_stats to Go

2019-09-27 Thread GitBox
ocket commented on a change in pull request #3758: Rewrote deliveryservice_stats to Go URL: https://github.com/apache/trafficcontrol/pull/3758#discussion_r329238877 ## File path: traffic_ops/traffic_ops_golang/trafficstats/trafficstats.go ## @@ -0,0 +1,691 @@ +package

[GitHub] [trafficcontrol] ocket8888 commented on a change in pull request #3758: Rewrote deliveryservice_stats to Go

2019-09-27 Thread GitBox
ocket commented on a change in pull request #3758: Rewrote deliveryservice_stats to Go URL: https://github.com/apache/trafficcontrol/pull/3758#discussion_r329206873 ## File path: traffic_ops/traffic_ops_golang/trafficstats/trafficstats.go ## @@ -0,0 +1,691 @@ +package

[GitHub] [trafficcontrol] ocket8888 commented on a change in pull request #3758: Rewrote deliveryservice_stats to Go

2019-09-27 Thread GitBox
ocket commented on a change in pull request #3758: Rewrote deliveryservice_stats to Go URL: https://github.com/apache/trafficcontrol/pull/3758#discussion_r32916 ## File path: traffic_ops/traffic_ops_golang/api/api.go ## @@ -369,6 +382,78 @@ func (inf *APIInfo)

[GitHub] [trafficcontrol] ocket8888 commented on a change in pull request #3758: Rewrote deliveryservice_stats to Go

2019-09-27 Thread GitBox
ocket commented on a change in pull request #3758: Rewrote deliveryservice_stats to Go URL: https://github.com/apache/trafficcontrol/pull/3758#discussion_r329147414 ## File path: traffic_ops/traffic_ops_golang/api/api.go ## @@ -369,6 +382,78 @@ func (inf *APIInfo)

[GitHub] [trafficcontrol] ocket8888 commented on a change in pull request #3758: Rewrote deliveryservice_stats to Go

2019-09-26 Thread GitBox
ocket commented on a change in pull request #3758: Rewrote deliveryservice_stats to Go URL: https://github.com/apache/trafficcontrol/pull/3758#discussion_r328806777 ## File path: traffic_ops/traffic_ops_golang/trafficstats/trafficstats.go ## @@ -0,0 +1,691 @@ +package

[GitHub] [trafficcontrol] ocket8888 commented on a change in pull request #3758: Rewrote deliveryservice_stats to Go

2019-09-26 Thread GitBox
ocket commented on a change in pull request #3758: Rewrote deliveryservice_stats to Go URL: https://github.com/apache/trafficcontrol/pull/3758#discussion_r328804291 ## File path: traffic_ops/traffic_ops_golang/trafficstats/trafficstats.go ## @@ -0,0 +1,691 @@ +package

[GitHub] [trafficcontrol] ocket8888 commented on a change in pull request #3758: Rewrote deliveryservice_stats to Go

2019-09-26 Thread GitBox
ocket commented on a change in pull request #3758: Rewrote deliveryservice_stats to Go URL: https://github.com/apache/trafficcontrol/pull/3758#discussion_r328802230 ## File path: traffic_ops/traffic_ops_golang/api/api.go ## @@ -369,6 +382,78 @@ func (inf *APIInfo)

[GitHub] [trafficcontrol] ocket8888 commented on a change in pull request #3758: Rewrote deliveryservice_stats to Go

2019-09-26 Thread GitBox
ocket commented on a change in pull request #3758: Rewrote deliveryservice_stats to Go URL: https://github.com/apache/trafficcontrol/pull/3758#discussion_r328801401 ## File path: traffic_ops/traffic_ops_golang/trafficstats/trafficstats.go ## @@ -0,0 +1,691 @@ +package

[GitHub] [trafficcontrol] ocket8888 commented on a change in pull request #3758: Rewrote deliveryservice_stats to Go

2019-09-26 Thread GitBox
ocket commented on a change in pull request #3758: Rewrote deliveryservice_stats to Go URL: https://github.com/apache/trafficcontrol/pull/3758#discussion_r328800927 ## File path: traffic_ops/traffic_ops_golang/trafficstats/trafficstats.go ## @@ -0,0 +1,691 @@ +package

[GitHub] [trafficcontrol] ocket8888 commented on a change in pull request #3758: Rewrote deliveryservice_stats to Go

2019-09-26 Thread GitBox
ocket commented on a change in pull request #3758: Rewrote deliveryservice_stats to Go URL: https://github.com/apache/trafficcontrol/pull/3758#discussion_r328800089 ## File path: traffic_ops/traffic_ops_golang/trafficstats/trafficstats.go ## @@ -0,0 +1,691 @@ +package

[GitHub] [trafficcontrol] ocket8888 commented on a change in pull request #3758: Rewrote deliveryservice_stats to Go

2019-09-26 Thread GitBox
ocket commented on a change in pull request #3758: Rewrote deliveryservice_stats to Go URL: https://github.com/apache/trafficcontrol/pull/3758#discussion_r328799708 ## File path: traffic_ops/traffic_ops_golang/trafficstats/trafficstats.go ## @@ -0,0 +1,691 @@ +package

[GitHub] [trafficcontrol] ocket8888 commented on a change in pull request #3758: Rewrote deliveryservice_stats to Go

2019-09-26 Thread GitBox
ocket commented on a change in pull request #3758: Rewrote deliveryservice_stats to Go URL: https://github.com/apache/trafficcontrol/pull/3758#discussion_r328793518 ## File path: traffic_ops/traffic_ops_golang/trafficstats/trafficstats.go ## @@ -0,0 +1,691 @@ +package

[GitHub] [trafficcontrol] ocket8888 commented on a change in pull request #3758: Rewrote deliveryservice_stats to Go

2019-09-26 Thread GitBox
ocket commented on a change in pull request #3758: Rewrote deliveryservice_stats to Go URL: https://github.com/apache/trafficcontrol/pull/3758#discussion_r328790159 ## File path: lib/go-rfc/mimetype_test.go ## @@ -0,0 +1,359 @@ +package tc Review comment: whoops

[GitHub] [trafficcontrol] ocket8888 commented on a change in pull request #3758: Rewrote deliveryservice_stats to Go

2019-09-26 Thread GitBox
ocket commented on a change in pull request #3758: Rewrote deliveryservice_stats to Go URL: https://github.com/apache/trafficcontrol/pull/3758#discussion_r328787404 ## File path: traffic_ops/traffic_ops_golang/trafficstats/trafficstats.go ## @@ -0,0 +1,691 @@ +package

[GitHub] [trafficcontrol] ocket8888 commented on a change in pull request #3758: Rewrote deliveryservice_stats to Go

2019-09-26 Thread GitBox
ocket commented on a change in pull request #3758: Rewrote deliveryservice_stats to Go URL: https://github.com/apache/trafficcontrol/pull/3758#discussion_r328783878 ## File path: traffic_ops/traffic_ops_golang/trafficstats/trafficstats.go ## @@ -0,0 +1,691 @@ +package

[GitHub] [trafficcontrol] ocket8888 commented on a change in pull request #3758: Rewrote deliveryservice_stats to Go

2019-09-26 Thread GitBox
ocket commented on a change in pull request #3758: Rewrote deliveryservice_stats to Go URL: https://github.com/apache/trafficcontrol/pull/3758#discussion_r328782937 ## File path: traffic_ops/traffic_ops_golang/trafficstats/trafficstats.go ## @@ -0,0 +1,691 @@ +package

[GitHub] [trafficcontrol] ocket8888 commented on a change in pull request #3758: Rewrote deliveryservice_stats to Go

2019-09-26 Thread GitBox
ocket commented on a change in pull request #3758: Rewrote deliveryservice_stats to Go URL: https://github.com/apache/trafficcontrol/pull/3758#discussion_r328777056 ## File path: traffic_ops/traffic_ops_golang/config/config.go ## @@ -122,6 +125,14 @@ type ConfigLDAP

[GitHub] [trafficcontrol] ocket8888 commented on a change in pull request #3758: Rewrote deliveryservice_stats to Go

2019-09-26 Thread GitBox
ocket commented on a change in pull request #3758: Rewrote deliveryservice_stats to Go URL: https://github.com/apache/trafficcontrol/pull/3758#discussion_r328776808 ## File path: traffic_ops/traffic_ops_golang/api/api.go ## @@ -369,6 +382,78 @@ func (inf *APIInfo)

[GitHub] [trafficcontrol] ocket8888 commented on a change in pull request #3758: Rewrote deliveryservice_stats to Go

2019-09-26 Thread GitBox
ocket commented on a change in pull request #3758: Rewrote deliveryservice_stats to Go URL: https://github.com/apache/trafficcontrol/pull/3758#discussion_r328774906 ## File path: traffic_ops/traffic_ops_golang/api/api.go ## @@ -369,6 +382,78 @@ func (inf *APIInfo)

[GitHub] [trafficcontrol] ocket8888 commented on a change in pull request #3758: Rewrote deliveryservice_stats to Go

2019-09-26 Thread GitBox
ocket commented on a change in pull request #3758: Rewrote deliveryservice_stats to Go URL: https://github.com/apache/trafficcontrol/pull/3758#discussion_r328771922 ## File path: lib/go-tc/traffic_stats.go ## @@ -0,0 +1,148 @@ +package tc + +/* + * Licensed to the

[GitHub] [trafficcontrol] ocket8888 commented on a change in pull request #3758: Rewrote deliveryservice_stats to Go

2019-09-26 Thread GitBox
ocket commented on a change in pull request #3758: Rewrote deliveryservice_stats to Go URL: https://github.com/apache/trafficcontrol/pull/3758#discussion_r328772150 ## File path: lib/go-tc/traffic_stats.go ## @@ -0,0 +1,148 @@ +package tc + +/* + * Licensed to the

[GitHub] [trafficcontrol] ocket8888 commented on a change in pull request #3758: Rewrote deliveryservice_stats to Go

2019-09-26 Thread GitBox
ocket commented on a change in pull request #3758: Rewrote deliveryservice_stats to Go URL: https://github.com/apache/trafficcontrol/pull/3758#discussion_r328771202 ## File path: lib/go-tc/traffic_stats.go ## @@ -0,0 +1,148 @@ +package tc + +/* + * Licensed to the

[GitHub] [trafficcontrol] ocket8888 commented on a change in pull request #3758: Rewrote deliveryservice_stats to Go

2019-07-26 Thread GitBox
ocket commented on a change in pull request #3758: Rewrote deliveryservice_stats to Go URL: https://github.com/apache/trafficcontrol/pull/3758#discussion_r307932576 ## File path: traffic_ops/traffic_ops_golang/routing/routes.go ## @@ -104,6 +105,9 @@ func Routes(d

[GitHub] [trafficcontrol] ocket8888 commented on a change in pull request #3758: Rewrote deliveryservice_stats to Go

2019-07-26 Thread GitBox
ocket commented on a change in pull request #3758: Rewrote deliveryservice_stats to Go URL: https://github.com/apache/trafficcontrol/pull/3758#discussion_r307908270 ## File path: traffic_ops/traffic_ops_golang/routing/routes.go ## @@ -104,6 +105,9 @@ func Routes(d