News for hackers. Stuff that matters.
Re:Go will use pdqsort in next release by dehrmann on Thursday April 21, 2022 @02:23AM (#31106514)
Is it stable, but not guaranteed to be stable, or is it unstable? If it just happens to be stable, code somewhere will eventually depend on that behavior.
Re:Go will use pdqsort in next release by infogulch on Thursday April 21, 2022 @03:25AM (#31106828)
Go has moved in this direction before, when they made map iteration order actually random to match the spec that said that you can't depend on iteration order. IIRC the impl is actually pretty light and only randomizes the starting index, and only generates the random offset once at startup, but it's enough to prevent people from accidentally depending on the order e.g. in their test suite. There might be a bit of disruption, but I expect it will be reasonably well received by most Go users.

There's a debate/lesson in here about when implementations should change to actually take up all the space claimed by the spec.

Re:Go will use pdqsort in next release by Cthulhu_ on Thursday April 21, 2022 @03:32AM (#31106861)
I kinda like that, because a lot of people will not read the spec but just try something first. If their map iteration is the same as e.g. insertion order, they will shrug and assume it's correct and move on.
Re:Go will use pdqsort in next release by tialaramex on Thursday April 21, 2022 @05:12AM (#31107369)
Go's decision here also defeats the next lazy thing people do, which is they run it once, and then they copy-paste whatever the results are as "correct" into their test case. With the randomisation their next run is also randomised and this "correct" answer will now be wrong so their tests fail again.

As a result most people will go "Oh, I guess I should actually test the outcome I care about" (good) or "Oh, testing is hard, I'll skip it" (bad, but at least not a problem for the team maintaining the standard library). Very few will say to themselves "I bet I can defeat the randomisation somehow with a different incorrect test design" because humans are lazy.

Re:Go will use pdqsort in next release by setr on Thursday April 21, 2022 @02:40AM (#31106599)
If you ignore the stated guarantees of the API and depend on implementation details instead, then it’s on you? I might understand if it were unstated, but it explicitly states it’s not.

Otherwise the only way to have a non-stable sort function that currently happens-to-be stable would have to include a randomized shuffle afterwards to ensure no one goes around depending on it…

It would be equivalent to assuming the golang ABI was stable because it survived unmodified for a couple versions, despite it being explicitly not. And then freezing the ABI to maintain a guarantee it explicitly didn’t want to maintain.

Re:Go will use pdqsort in next release by Someone on Thursday April 21, 2022 @03:37AM (#31106888)
> if you ignore the stated guarantees of the API and depend on implementation details instead, then it’s on you?

In theory, yes. In practice, if a OS or library update uncovers bugs in many programs, that update won’t get favorable reviews.

That’s the reason behind Linux’s “we do not break userspace!” rule (https://linuxreviews.org/WE_DO_NOT_BREAK_USERSPACE), and also why Microsoft Windows has layer upon layer of APIs, a copy of the Windows 95 memory manager (https://mcpmag.com/articles/2010/07/27/trip-down-memory-lane...), why large databases support character sets that probably haven’t been used anywhere since before many HN readers were born (https://docs.oracle.com/goldengate/1212/gg-winux/GWUAD/wu_ch...), etc.

(Apple has a totally different viewpoint on this. That’s one reason it never really got into businesses for a long time)

Re:Go will use pdqsort in next release by samus on Thursday April 21, 2022 @01:16PM (#31112313)
Linus' rule also has disadvantages as it blocks real innovation in the Linux ecosystem. Granted, Desktop Linux doesn't need another Python 2 -> 3 or X.org -> Wayland deathmarch...
Re:Go will use pdqsort in next release by ElectricalUnion on Thursday April 21, 2022 @09:22PM (#31117190)
> Linus' rule also has disadvantages as it blocks real innovation in the Linux ecosystem.

Block real innovation like what?

> Granted, Desktop Linux doesn't need another Python 2 -> 3 or X.org -> Wayland deathmarch...

That's all userspace? Userspace doesn't care it breaks. That's why userspace sucks. You need to keep compatibility if you want to have a stable base to build something. That's why we have (sometimes silly) standards like SUS or POSIX.

"we do not break userspace" at least garantees that you need to fix only the userspace, and not the whole thing all the time.

Re:Go will use pdqsort in next release by samus on Friday April 22, 2022 @03:50AM (#31119270)
Breakages of userspace are almost always unintended consequences. I don't have examples to the contrary, but I believe that intentional breakages and redesigns are almost unthinkable because of Linus's rule.

Significant changes in the kernel already often need years to land because of the multitude of configurations and interfaces that have to be supported. And even longer to deprecate and remove. Not necessarily a bad thing, just the price of success.

Large-scale breakages like the Wayland transition affect the whole Desktop Linux ecosystem, and are just an example of what mayhem fundamental changes can cause.

Re:Go will use pdqsort in next release by remus on Thursday April 21, 2022 @03:32AM (#31106859)
While I totally agree with the sentiment it is nice when language developers go out of their way to not break stuff, even when they don't have to. For example the golang team have made efforts not to break certain usages of the unsafe package in the past, despite the package offering no guarantees about stability.

On a similar note, Im sure there was a suggestion to artificially make the previous golang sort function unstable so that users wouldn't come to rely on stable behaviour where it wasn't guaranteed. I quite like the idea, start the fight against hyrum's law!

Re:Go will use pdqsort in next release by edflsafoiewq on Thursday April 21, 2022 @02:39AM (#31106592)
Happened in JS land. IE was stable and sites depended on it. FF switched to match IE. Chrome switched to match IE and FF. And then finally they put it into the spec.
Re:Go will use pdqsort in next release by ElectricalUnion on Thursday April 21, 2022 @09:24PM (#31117210)
> IE was stable

[citation needed]

This is the wrong wording, IE had a decently enough installed/captive user base. It was never particularly stable.

Re:Go will use pdqsort in next release by dspillett on Thursday April 21, 2022 @04:04AM (#31107023)
The former. The spec just says the result will be sorted, it just happens that the current implementations use a method that results in stable positioning of items with equal sort value.

There aren't a lot of places where stability matters, where it does the programmer should hopefully know and not have relied on an undefined behaviour… The most likely impact, I think, will be in UIs where the difference won't really matter but could be quite noticeable.

Do note though that "unstable" can mean two different things. A single threaded sort will produce the same result for the same input, and many parallels sorts have this property too due to the way they coordinate the threads and collate their results, so the output order of equal items is arbitrary but not random. Some parallel sorts do order items with the same sort key more randomly though due to being sensitive to timing differences caused by uncontrolled factors (the OS's scheduler and other things it is dealing with, differing IO delays if the sort is not in-memory, different numbers of threads if comparing run results with those done on other processor architectures, …). IIRC pdqsort is the former if those two types of unstable.

Re:Go will use pdqsort in next release by tgv on Thursday April 21, 2022 @02:43AM (#31106618)
Good question. I checked the 1.18.1 code, and Sort() and Stable() call different functions. The former calls quick sort, the latter something else (groups of 20 items with insertion sort, then symmerge?). Quick sort's traditional implementation isn't stable, so I'd be surprised if Sort() would actually be.