[GitHub] [trafficcontrol] ocket8888 commented on issue #3880: Rewrote /capabilities/{{name}} to Go

2019-10-10 Thread GitBox
ocket commented on issue #3880: Rewrote /capabilities/{{name}} to Go
URL: https://github.com/apache/trafficcontrol/pull/3880#issuecomment-540877296
 
 
   I don't think so. Making changes in Go is easier than in Perl, so if 
somebody wants to spend the time writing it and someone else wants to spend the 
time reviewing it, a rewrite of a deprecated endpoint could still be valuable. 
It lets Perl die faster. Worst case scenario, we've wasted some time, but the 
impact on the project is neutral - not strictly negative.


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


With regards,
Apache Git Services


[GitHub] [trafficcontrol] ocket8888 commented on issue #3880: Rewrote /capabilities/{{name}} to Go

2019-10-09 Thread GitBox
ocket commented on issue #3880: Rewrote /capabilities/{{name}} to Go
URL: https://github.com/apache/trafficcontrol/pull/3880#issuecomment-540224884
 
 
   That thread never reached a decision as to whether or not deprecated routes 
ought to be rewritten. If a decision to not rewrite these handlers is reached 
I'll close this PR (assuming it's still open), in the meantime I guess it's in 
the hands of people browsing PRs whether or not this is worth merging (once it 
isn't a draft (if ever)).


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


With regards,
Apache Git Services