hongfeiweb commented on issue #2029: Implements https://github.com/apache/incubator-trafficcontrol/issues/… URL: https://github.com/apache/incubator-trafficcontrol/pull/2029#issuecomment-378308231 @DylanVolz - i am not convinced that moving the URL to top level (as cachegroup_fallbacks) for this functionality is the right direction. I think the guideline wiki cited a good use of toplevel for profile & parameters. In such case, the parameters and profiles are loosely associated where params don't necessarily get associated with any profile. The fallbacks however are different in that it's just some additional, optional attributes to an otherwise regular cachgroup therefore should be accessible only through ../cachgroup/... hierarchy ihmo. I don't think the database storage normalization has to necessarily influence the API representation.
---------------------------------------------------------------- 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
