Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

race test ConsoleLoggerTest > getConsoleLoggerShouldBeThreadSafe() hangs on JDK21 and new ThreadPerTaskBoundedElasticScheduler #3628

Closed
OlegDokuka opened this issue Nov 2, 2023 · 0 comments · Fixed by #3678
Closed

race test ConsoleLoggerTest > getConsoleLoggerShouldBeThreadSafe() hangs on JDK21 and new ThreadPerTaskBoundedElasticScheduler #3628

OlegDokuka opened this issue Nov 2, 2023 · 0 comments · Fixed by #3678
Labels
type/bug A general bug type/test/flaky

Comments

@OlegDokuka
Copy link
Contributor 

ConsoleLoggerTest > getConsoleLoggerShouldBeThreadSafe() FAILED
    java.lang.AssertionError: RaceTestUtils.race wait timed out after 5s
        at reactor.test.util.RaceTestUtils.race(RaceTestUtils.java:180)
        at reactor.test.util.RaceTestUtils.race(RaceTestUtils.java:144)
        at reactor.test.util.RaceTestUtils.race(RaceTestUtils.java:108)
        at reactor.util.ConsoleLoggerTest.getConsoleLoggerShouldBeThreadSafe(ConsoleLoggerTest.java:319)
@reactorbot reactorbot added the ❓need-triage This issue needs triage, hasn't been looked at by a team member yet label Nov 2, 2023
@OlegDokuka OlegDokuka changed the title race test ConsoleLoggerTest > getConsoleLoggerShouldBeThreadSafe() fails on JDK21 and new ThreadPerTaskBoundedElasticScheduler race test ConsoleLoggerTest > getConsoleLoggerShouldBeThreadSafe() hangs on JDK21 and new ThreadPerTaskBoundedElasticScheduler Nov 2, 2023
@OlegDokuka OlegDokuka added type/bug A general bug and removed ❓need-triage This issue needs triage, hasn't been looked at by a team member yet labels Nov 2, 2023
@chemicL chemicL mentioned this issue Jan 5, 2024
Merged
chemicL added a commit that referenced this issue Jan 5, 2024
@chemicL
Improve RaceTestUtils to yield when CPUs don't suffice (#3678)
292b21b 
When `RaceTestUtils`-based concurrency tests involving more tasks than
available CPUs were run, they'd time-out when run in the Java 21 setup
with loom-based boundedElastic `Scheduler`. This change should eliminate
such situations.

Addresses related issues below and potentially more.
Fixes #3629
Fixes #3628
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
type/bug A general bug type/test/flaky
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Improve RaceTestUtils to yield when CPUs don't suffice reactor/reactor-core
2 participants
@OlegDokuka @reactorbot