rawlinp commented on a change in pull request #2785: In Traffic Router Support 
Snapshots which only update Delivery Services
URL: https://github.com/apache/trafficcontrol/pull/2785#discussion_r236449245
 
 

 ##########
 File path: 
traffic_router/core/src/main/java/com/comcast/cdn/traffic_control/traffic_router/core/cache/Cache.java
 ##########
 @@ -187,6 +210,29 @@ public String getDeliveryServiceId() {
                public String getFqdn() {
                        return fqdn;
                }
+
+               /*
+                * Aliases are only used to create and update the statTracker
+                */
+               public List<String> getAliases() {
+                       return aliases;
+               }
+
+               public void setAliases( final List<String> aliasesLst) {
 
 Review comment:
   Can you remove the space before the `final`? Also, do you mind renaming 
`aliasesLst` to `aliasesList` here?

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to