rob05c commented on issue #2274: Profiles endpoint unordered
URL: 
https://github.com/apache/incubator-trafficcontrol/issues/2274#issuecomment-390024452
 
 
   Just because something is common doesn't make it good. And yes, we have it 
on existing endpoints. But IMO we should be reducing unnecessary server load, 
not increasing it.
   
   In this particular case, Perl didn't "support" it, it was an incidental 
implementation. Right now, things like this aren't a big deal, because we have 
a very slow requests/second rate, due to changes being limited to operations. 
But as we move to Self Service, that's going to change, and load is going to 
start mattering more, as hundreds and even thousands of users are using Traffic 
Ops at once, and each user is changing configuration much more frequently.
   
   When that happens, there's a big difference in the TO server having to 
process 1000 concurrent requests and do the extra work to sort all of them at 
the same time, versus having all 1000 users sort their data client-side in the 
browser.
   
   Ordering one endpoint is small, but every decision we make like this affects 
the overall scalability of the application. We should make good decisions now, 
not putting unnecessary work on the server, so we don't have to do more work to 
refactor both the client and the server, when Self Service increases our load 
1000x.

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


With regards,
Apache Git Services

Reply via email to