[GitHub] [trafficcontrol] rawlinp commented on issue #4303: Updated GetDeliveryServiceByXMLID function to return a DS nullable struct

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

rawlinp commented on issue #4303: Updated GetDeliveryServiceByXMLID function to return a DS nullable struct
URL: https://github.com/apache/trafficcontrol/pull/4303#issuecomment-575260922
 
 
   Yeah, in addition to the mailing list discussion I'd also add that going the deprecation route would be great if we actually cleaned up deprecated functions, stopped increasing the usage of already-deprecated functions, etc. But historically nobody has really gone back to clean things up, and I've definitely encountered new code that used already-deprecated client methods. At least if you make a breaking change to the TO Go client you're forced to fix all the existing usages in the repo (otherwise the build fails), and we don't end up w/ 3 different client methods that all do the same thing.

----------------------------------------------------------------
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