News for hackers. Stuff that matters.
Re:Tell HN: Thank you for not redesigning Hacker News by Deimorz on Sunday September 01, 2019 @06:42PM (#20855491)
Sure, of course it can be much worse, and almost all other sites are. I'm just saying that HN could do it better, because there's no need for it to send a redirect or that full-page response unless the user has JS disabled.

With JS, they could send a completely empty HTTP 200 response and it would work exactly the same (it does nothing with the response anyway). Right now HN is probably sending multiple gigabytes worth of responses to voting every day that are all just thrown away.

Re:Tell HN: Thank you for not redesigning Hacker News by noobermin on Sunday September 01, 2019 @09:03PM (#20856003)
>almost all other sites are.

That's their point. You're not wrong, you can always be better, but HN is barebones compared to almost all (using the mathematical definition...alright, facetiously using it) other sites.

Re:Tell HN: Thank you for not redesigning Hacker News by drusepth on Monday September 02, 2019 @12:12AM (#20856724)
Comparing to Facebook is probably the worst possible comparison, though, and doesn't seem representative of most other sites.
Re:Tell HN: Thank you for not redesigning Hacker News by GoblinSlayer on Monday September 02, 2019 @05:01AM (#20857751)
Only a very few sites are not bloated Facebook clone abominations.
Re:Tell HN: Thank you for not redesigning Hacker News by netfl0 on Sunday September 01, 2019 @10:39PM (#20856393)
He is wrong. It is not “quite bad”.

I appreciate your generosity to him but I am sad to see that is the top comment.

Re:Tell HN: Thank you for not redesigning Hacker News by Deimorz on Sunday September 01, 2019 @10:59PM (#20856476)
How could it possibly be considered a good thing? I've seen dang say before that they've been running into performance issues, and I bet that they'd see a significant drop in resource usage solely by not generating entire comments pages just to throw them away every time anyone votes on a comment. The response is literally not used at all, they're doing that work and sending that data for no reason.

There are multiple downsides and not a single benefit to the current method.

Re:Tell HN: Thank you for not redesigning Hacker News by samtheprogram on Sunday September 01, 2019 @11:55PM (#20856668)
It's implemented this way for balance of a simple implementation and how lightweight it is. Not to mention, there is at least one benefit: the voting mechanism doesn't depend on JavaScript.

And if you are not authenticated, you are instead taken to a login page. Logging in then gives you the redirect and the upvote is registered.

Re:Tell HN: Thank you for not redesigning Hacker News by 171243 on Monday September 02, 2019 @12:28AM (#20856775)
So just to clarify, this behavior seems to be by design. In that case it does seem the original comment is objectively wrong if the code is readable and maintainable and accomplishes the desired behavior.
Re:Tell HN: Thank you for not redesigning Hacker News by csande17 on Monday September 02, 2019 @01:26AM (#20856947)
As parent comments mention, sending back the whole-page response when you vote on a comment is a great idea if you have JavaScript disabled or aren't logged in.

But it'd be a quick performance win if it didn't do that when you were logged in with JavaScript enabled. The actual JavaScript part of this is actually already implemented -- that's why the page doesn't refresh when you upvote my comment -- so you'd just need one if-statement on the server side to send a blank response instead of a redirect.

Sure, 10 KB of download and the server CPU time needed to re-generate the listing page is probably pretty negligible, but multiply that by the number of votes casted on Hacker News every day and it starts to add up.

Re:Tell HN: Thank you for not redesigning Hacker News by Deimorz on Monday September 02, 2019 @01:45AM (#20857013)
Exactly, and it even adds up quickly for a single user in common cases like voting on a bunch of comments while reading a thread.

If someone loads this thread right now, the initial page load is 61KB (gzipped). If they read down the thread and vote on 30 comments while they're doing it, they transfer an additional 480KB (16KB/vote [1]). There's no reason that they should be receiving 8x more data by voting on a relatively low number of comments (~10% of the thread) than they did when loading the entire page originally. It also has to re-generate the page an extra 30 times unnecessarily - a repeatedly voting user becomes like a heavy multiplier to your traffic.

[1]: Also, I just noticed that the responses back from voting actually aren't the full page, they're truncated. It seems like maybe the response size is capped at 16KB gzipped? That implies that there's already some kind of special handling for the voting responses.

Re:Tell HN: Thank you for not redesigning Hacker News by southerntofu on Monday September 02, 2019 @05:31AM (#20857876)
From what i can see, the response only contains the comment you upvoted and its children. It's important for people like me who don't have Javascript, but the JS voting system could probably do with a simple HEAD request.
Re:Tell HN: Thank you for not redesigning Hacker News by TeMPOraL on Monday September 02, 2019 @03:13AM (#20857300)
Seconding. Sending back the whole page on AJAX upvote seems unnecessarily wasteful, and sounds like something trivially fixable.

Taking data from 2.5 years ago[0], HN has ~300k daily uniques. Assuming 10% of that are users, each upvoting one story, that's 30k unnecessary page renders per day. That's ~2% of the total views (though the more expensive views - AFAIK when you're logged out, you get cached pages?), so on the one hand it's not much, but on the other hand the fix should be trivial and have minimal impact on code readability.

--

[0] - https://news.ycombinator.com/item?id=9220098

Re:Tell HN: Thank you for not redesigning Hacker News by zzo38computer on Monday September 02, 2019 @04:58PM (#20861725)
It is possible to do both. Such as, have the link like it is now, but add a JavaScript code which replaces the link with one that executes the JavaScript code to make a request that returns nothing (if you are logged in). If JavaScript is disabled or unimplemented, it will still work.
Re:Tell HN: Thank you for not redesigning Hacker News by jbverschoor on Monday September 02, 2019 @03:21AM (#20857331)
You can return NO_CONTENT on a successful request, and the browser will do nothing.
Re:Tell HN: Thank you for not redesigning Hacker News by roenxi on Monday September 02, 2019 @03:22AM (#20857334)
> How could it possibly be considered a good thing?

It will work with any possible browser under any reasonably foreseeable condition with no maintenance on the part of the HN crew.

> I bet that they'd see a significant drop in resource usage solely by not generating entire comments pages just to throw them away every time anyone votes on a comment.

If there is a serious performance problem they will presumably do some analysis and make changes related to the most serious bottleneck.

> There are multiple downsides

There are multiple irrelevant hypotheticals. If anyone cares about the heavy-weight HN, I hope they don't accidentally click the link to the article and discover the wider internet with images and whatnot.

Re:Tell HN: Thank you for not redesigning Hacker News by neop1x on Monday September 02, 2019 @05:38AM (#20857903)
True. And I can see that some webs gradually stop working if you are using JS blockers. For example Aliexpress used to work fine with uMatrix. Now, if I click on the Next page in the paginator, it behaves like it loaded the next page, jumps to top, but it didn't - it just changed URL location. So I have to CTRL+R to reload the whole page to force their server-side rendering. I know it is theoretically my problem but I have to use JS blocking to stop (for me useless) ads and malware scripts. If they just loaded content from their domain or sub-domains, it would work perfectly fine. And if it were old-school server-rendered website, it would probably work faster, as it was discussed here some time ago. I have a feeling that web devs are sometimes over-engineering it. Web browser is HTML viewer, not an operating system.