[GitHub] [trafficcontrol] ocket8888 edited a comment on issue #3075: Traffic Ops Golang parent.config

[GitBox <gi...@apache.org>]

ocket8888 edited a comment on issue #3075: Traffic Ops Golang parent.config
URL: https://github.com/apache/trafficcontrol/pull/3075#issuecomment-482692769
 
 
   So I'm still seeing some weird diffs
   ```diff
   1,2c1,2
   < # DO NOT EDIT - Generated for REDACTED by Traffic Ops (https://REDACTED/) on Fri Apr 12 17:00:47 UTC 2019
   < dest_domain=REDACTED port=80 parent="REDACTED_A:80|1.0;REDACTED_B:80|1.0;" round_robin=consistent_hash qstring=consider go_direct=false parent_is_proxy=false parent_retry=both max_simple_retries=1 max_unavailable_server_retries=1
   ---
   > # DO NOT EDIT - Generated for REDACTED by Traffic Ops (https://REDACTED/) on Fri Apr 12 17:00:43 UTC 2019
   > dest_domain=REDACTED port=80 parent="REDACTED_B:80|1.0;REDACTED_A:80|1.0;" round_robin= qstring=ignore go_direct=false parent_is_proxy=false
   ```
   You'll notice that the parent ordering is different, but since they're generated with explicit quality params, it shouldn't (significantly) alter ATS behavior. The discrepancy between which fields are present is probably a known issue that I like to call The Duplicate Origin Problem<sup>TM</sup> - though that's probably a bad name. However, what **is** a problem is `round_robin=` with no value. Need to figure that out.

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services