Discovering a JDK Race Condition, and Debugging It in 30 Minutes with Fray
108 points
15 hours ago
| 4 comments
| aoli.al
| HN
exabrial
6 hours ago
[-]
> Fray is a concurrency testing tool for Java that can help you find and debug tricky race conditions that manifest as assertion violations, run-time exceptions, or deadlocks. It performs controlled concurrency testing using state-of-the-art techniques such as probabilistic concurrency testing or partial order sampling.

> Fray also provides deterministic replay capabilities for debugging specific thread interleavings. Fray is designed to be easy to use and can be integrated into existing testing frameworks.

I wish I had this 20 years ago.

reply
MaxBarraclough
13 hours ago
[-]
Neat to see sleep calls artificially introduced to reliably recreate the deadlock. [0]

Looks like fixing the underlying bug is still in-progress, [1] I wonder how many lines of code it will take.

[0] https://github.com/aoli-al/jdk/commit/625420ba82d2b0ebac24d9...

[1] https://bugs.openjdk.org/browse/JDK-8358601

reply
brabel
1 hour ago
[-]
Bugs like these are pervasive in languages like Java that give no protection against even the most basic race condition causes. It’s nearly impossible to write reliable concurrent code. Freya only helps if you actually use it to test everything which is not realistic. I am convinced, after my last year long struggle to get a highly concurrent Java (actually Kotlin but Kotlin does not add much to help) module at work, that we should only use languages that provide safe concurrency models, like Erlang/Elixir and Rust, or actor-like like Dart and JavaScript, where concurrency is required.
reply
trhway
6 hours ago
[-]
without reworking of the code all these checks of the executor and queue state and queue manipulations have to be under a mutex, and that is just a few lines.
reply
latchkey
14 hours ago
[-]
Maybe it is just me, but I can't read the text in the code because the font is nearly white on white.
reply
masklinn
14 hours ago
[-]
The light mode is fine, but you're right the dark mode is truly awful, the code blocks are unreadable.

edit: for some reason the author overrode the background color on code blocks via an inline style of

    background-color:#f0f0f0
from

    var(--code-background-color) = #f2f2f2
to make the background nigh imperceptibly darker, but then while the stylesheet properly switches the to #01242e in dark mode the inline override stays and blows it to bit.

Not that it's amazing if you remove the inline stle, on account of operators and method names being styled pretty dark (#666 and #4070a0).

reply
aoli-al
14 hours ago
[-]
Thanks for pointing it out! Just did a quick fix using Claude :)
reply
malcolmgreaves
13 hours ago
[-]
On mobile (Safari), the lines in the code blocks have different font sizes. They also have different fonts. Some are like 3-4x the size of other lines. No idea what could be going wrong, but it does unfortunately make the code blocks difficult to follow along.
reply
aoli-al
13 hours ago
[-]
should be fixed as well :)
reply
NooneAtAll3
12 hours ago
[-]
any chance you can make light/dark mode switch a UI button?
reply
masklinn
4 hours ago
[-]
On desktop I’d suggest installing an extension that adds a toggle (they exist for Firefox and chrome at least): adding a toggle manually is a bit of a chore, especially if the css system you use does not build that in.
reply
TYMorningCoffee
9 hours ago
[-]
Impressive! Can't wait to try Fray out at work.
reply