Skip to content

Java transitive search performance improvements#155

Merged
zvigrinberg merged 3 commits into
RHEcosystemAppEng:rh-aiq-mainfrom
tmihalac:java-transitive-search-performance-improvments
Dec 3, 2025
Merged

Java transitive search performance improvements#155
zvigrinberg merged 3 commits into
RHEcosystemAppEng:rh-aiq-mainfrom
tmihalac:java-transitive-search-performance-improvments

Conversation

@tmihalac

@tmihalac tmihalac commented Dec 2, 2025

Copy link
Copy Markdown
Collaborator

No description provided.

Signed-off-by: Theodor Mihalache <tmihalac@tmihalac-thinkpadp1gen7.rmtusfl.csb>
@tmihalac tmihalac requested a review from zvigrinberg December 2, 2025 18:10

@zvigrinberg zvigrinberg left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

@tmihalac
I Think that you forgot to "revive" the standard lib test ( that triggered the performance issue thing.)

@tmihalac

tmihalac commented Dec 3, 2025

Copy link
Copy Markdown
Collaborator Author

@tmihalac I Think that you forgot to "revive" the standard lib test ( that triggered the performance issue thing.)

I didn't, it still can run for a very long time depending on which code path is randomly chosen, I can tell you that in 1 case it ran an hour and 5 min with the performance improvements

@zvigrinberg

Copy link
Copy Markdown
Collaborator

@tmihalac I Think that you forgot to "revive" the standard lib test ( that triggered the performance issue thing.)

I didn't, it still can run for a very long time depending on which code path is randomly chosen, I can tell you that in 1 case it ran an hour and 5 min with the performance improvements

OK I See.

@zvigrinberg zvigrinberg left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

Signed-off-by: Theodor Mihalache <tmihalac@tmihalac-thinkpadp1gen7.rmtusfl.csb>
@tmihalac

tmihalac commented Dec 3, 2025

Copy link
Copy Markdown
Collaborator Author

Hi, I pushed a fix to a bug I encountered, I introduced in the previous push

Signed-off-by: Theodor Mihalache <tmihalac@tmihalac-thinkpadp1gen7.rmtusfl.csb>
@zvigrinberg zvigrinberg merged commit e132920 into RHEcosystemAppEng:rh-aiq-main Dec 3, 2025
1 check passed
heatherzh01 pushed a commit that referenced this pull request Mar 6, 2026
…nce-improvments

fix: Java transitive search performance improvements
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants