Quarkus: Remove the workaround for ForkJoin to spin up threads

Created on 20 Nov 2019  路  7Comments  路  Source: quarkusio/quarkus

Describe the bug
We set -J-Djava.util.concurrent.ForkJoinPool.common.parallelism=1 in NativeImageBuildStep
to workaround for

It seems the GraalVM issue was resolved; are we safe to remove the workaround from Quarkus?

@stuartwdouglas needs your opinion at least

kinbug

All 7 comments

I'm not sure the commit has been included in 19.2.x:
https://github.com/oracle/graal/commit/e78fc21f53488848f791dca3db394031698177f2

From what I can see, it's only in 19.3 but I might be wrong.

This was included in 19.3.0 indeed.

This would be a good fix to have, I think it may speed up our native builds as I know the native compilation does use fork join internally in places.

This was included in 19.3.0 indeed.

@gwenneg Good to know. Any chance you could take this as part of your on going work with 19.3.0 ?

@machi1990 Sure, this could be part of #5358

Thanks @gwenneg

Done in #5358.

Was this page helpful?
0 / 5 - 0 ratings