Github user alficles commented on the issue:

    https://github.com/apache/incubator-trafficcontrol/pull/695
  
    This is pretty cool. It looks like there are two separate PRs in here, and 
the second one could really use a squash. The astats changes can stand on their 
own, so they should probably get their own PR. And the IPv6 for TM should be 
squashed. There are some confusing reverts of inadvertent changes that probably 
shouldn't go in, even as intermediate changesets.
    
    It'll be a lot easier to review it that way, I think.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Reply via email to