[GitHub] rawlinp commented on a change in pull request #2785: In Traffic Router Support Snapshots which only update Delivery Services

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

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_r236448376
 
 

 ##########
 File path: traffic_router/core/src/main/java/com/comcast/cdn/traffic_control/traffic_router/core/cache/Cache.java
 ##########
 @@ -142,6 +145,11 @@ public void setDeliveryServices(final Collection<DeliveryServiceReference> deliv
 		}
 	}
 
+	public void replaceDeliveryServices(final Collection<DeliveryServiceReference> newDeliveryServices){
+		this.deliveryServices.clear();
+		newDeliveryServices.forEach(dsr-> deliveryServices.put(dsr.getDeliveryServiceId(),dsr));
 
 Review comment:
   Can you add a space before the `->` and near the end before `dsr`?

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


With regards,
Apache Git Services