←back to thread

How I cut GTA Online loading times by 70%

(nee.lv)
3883 points kuroguro | 10 comments | 28 Feb 21 19:38 UTC | HN request time: 0.993s | source | bottom
Show context
capableweb ◴[28 Feb 21 20:18 UTC] No.26296659[source]▶
> the problems shouldn’t take more than a day for a single dev to solve

It bothers me that so many of us developers are annoyed by managers saying stupid stuff like "shouldn't be too hard" about things they don't understand, but then other developers walk into the same trap.

Yes, it looks simple at the surface. It most likely isn't. The problem is not that it looks simple, the problem is that you assume it is simple, probably because you don't have the full context.

replies(4): >>26296675 #>>26296687 #>>26296709 #>>26296737 #
1. asadlionpk ◴[28 Feb 21 20:19 UTC] No.26296675[source]▶
The person posted a PoC that works. Surely they have context now?
replies(2): >>26296683 #>>26301682 #
2. capableweb ◴[28 Feb 21 20:20 UTC] No.26296683[source]▶
That's very true, and the PoC works for that person. It's not that easy in real-life development though, you can't just switch out the JSON parser and call it a day. Lots of testing has to be done and you have to go through all the previous changes to make sure you're not missing what the previous maintainers did to fix some one-in-a-million issues.

I'm not saying it's impossible for this to be as easy as the author claims it to be. I'm just saying that it might not actually be that easy in reality, if you're on the inside.

replies(4): >>26296745 #>>26296746 #>>26297122 #>>26297316 #
3. jhncls ◴[28 Feb 21 20:27 UTC] No.26296745[source]▶
You seem to be hinting to Chesterton's fence [0].

[0]: https://fs.blog/2020/03/chestertons-fence/

replies(2): >>26296941 #>>26297749 #
4. asadlionpk ◴[28 Feb 21 20:27 UTC] No.26296746[source]▶
But you do understand the ROI of this relatively simple fix?
replies(1): >>26298265 #
5. resonantjacket5 ◴[28 Feb 21 20:50 UTC] No.26296941{3}[source]▶
Accept in this case I don't really see why you'd want a naive o(n^2) parser when it isn't needed. Well besides for slow loading times.
6. segmondy ◴[28 Feb 21 21:13 UTC] No.26297122[source]▶
Let's say that JSON parser is used in so many other places and we are afraid of regression. Guess what? We can make a new one. Let's call it newImprovedJsonParser(). Use that only at this spot, and you have fixed the issue without breaking other parts. You can then replace the other's once you vet that they are compatible.
7. toast0 ◴[28 Feb 21 21:37 UTC] No.26297316[source]▶
OK, don't switch out the JSON parser. Just patch in the caching strlen before parsing the json, and set it back to normal strlen after; and fiddle with the hashmap stuff.
8. asadlionpk ◴[28 Feb 21 22:34 UTC] No.26297749{3}[source]▶
Thanks for sharing this, didn't have a name for this scenario.
9. rvnx ◴[28 Feb 21 23:51 UTC] No.26298265{3}[source]▶
At such big amount of scale of revenues, I'm sure that GTA Online has feature flags that can act as canary deploys or A/B tests groups.

Deploy one hacked version (with the dirty fix / hack for this case) to one group, deploy the standard version to another group, measure the ROI and the crash-rate.

If the results are similar, then you validate permanently the fix and live happily after.

6 minutes to 1 minutes 10 is massive, it will even bring new players to join the game (I don't like to play GTA Online, though it's a great game, just because of loading times).

10. fctorial ◴[01 Mar 21 11:23 UTC] No.26301682[source]▶
This bug might be one of a two bug pair that cancel each other. Not very uncommon.