Skip to content

fix: resolve ProxyBuilderImpl ClassNotFoundException in parallel import#1288

Open
adhurjaty wants to merge 2 commits intoadorsys:mainfrom
adhurjaty:fix-class-loading-parallel
Open

fix: resolve ProxyBuilderImpl ClassNotFoundException in parallel import#1288
adhurjaty wants to merge 2 commits intoadorsys:mainfrom
adhurjaty:fix-class-loading-parallel

Conversation

@adhurjaty
Copy link

What this PR does / why we need it:

Fix Parallel import: ClassNotFoundException: org.jboss.resteasy.client.jaxrs.internal.proxy.ProxyBuilderImpl by explicitly setting the ClassLoader to match the main thread's in parallel forEach consumers.

Which issue this PR fixes: fixes #1107

Special notes for your reviewer:

PR Readiness Checklist:

Complete these before marking the PR as ready to review:

  • the CHANGELOG.md release notes have been updated to reflect any significant (and particularly user-facing) changes introduced by this PR

@adhurjaty adhurjaty force-pushed the fix-class-loading-parallel branch from 1d5d2f4 to b209afa Compare March 11, 2025 15:50
@sonarqubecloud
Copy link

@Motouom
Copy link
Collaborator

Motouom commented Jan 23, 2026

Hello @adhurjaty .

Please can you rebase to the main and make sure that your workflow passes?

The pipeline was fixed.

Thanks

@adhurjaty adhurjaty force-pushed the fix-class-loading-parallel branch 2 times, most recently from 35f8a02 to ad58759 Compare January 28, 2026 22:25
@Motouom
Copy link
Collaborator

Motouom commented Feb 23, 2026

Thank you much, @adhurjaty

.factorypath Outdated
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please can you remove this file @adhurjaty? It is an Eclipse-specific file generated

@adhurjaty adhurjaty force-pushed the fix-class-loading-parallel branch 5 times, most recently from 592a7c2 to 518264a Compare February 23, 2026 15:58
@adhurjaty adhurjaty requested a review from Motouom February 23, 2026 15:59
@Motouom
Copy link
Collaborator

Motouom commented Mar 5, 2026

Did you consider using the ThreadFactory because i think that the parallelStream will be a little problematic, I mean from my point of view.

Your implementation uses parallelStream(), which by default runs on the ForkJoinPool.commonPool(). If your consumer.accept(x) performs a blocking I/O operation (like a database call), it can starve the entire JVM. Every other part of your app using parallel streams will hang because the shared pool is exhausted.

@Motouom Motouom moved this from Review to Review Requested in os-competence-center-board Mar 9, 2026
adhurjaty and others added 2 commits March 9, 2026 13:27
@sonarqubecloud
Copy link

sonarqubecloud bot commented Mar 9, 2026

@Motouom Motouom self-requested a review March 23, 2026 10:24
@Motouom Motouom moved this from Changes Requested to Review in os-competence-center-board Mar 23, 2026
@Motouom Motouom moved this from Review to Changes Requested in os-competence-center-board Mar 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug enhancement java Pull requests that update Java code

Projects

Status: Changes Requested

Development

Successfully merging this pull request may close these issues.

Parallel import: ClassNotFoundException: org.jboss.resteasy.client.jaxrs.internal.proxy.ProxyBuilderImpl

4 participants