mattjackson220 commented on a change in pull request #5011:
URL: https://github.com/apache/trafficcontrol/pull/5011#discussion_r488822277
##########
File path:
traffic_router/core/src/main/java/com/comcast/cdn/traffic_control/traffic_router/core/router/StatTracker.java
##########
@@ -184,7 +184,13 @@ public void setRegionalAlternateCount(final int
regionalAlternateCount) {
/**
* This value indicates that routing was blocked in
accordance with anonymous blocking configurations.
*/
- ANON_BLOCK
+ ANON_BLOCK,
+ /**
+ * This value indicates that routing was based on the
default lat/long of the delivery service, because maxmind
+ * returned the centre of the country as the client
location, due to the CZF not being able to resolve the client IP
+ * to a valid location.
+ */
+ GEO_DS
Review comment:
yea thats why i was thinking it might be good to add another one in
there that tells us that it is just using the maxmind defaults. eh if theres
value there itll be an easy change to add in if people decide they want it
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
