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.

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

Add workaround for possible ApiTools leak #2963

Merged

Conversation

laeubi
Copy link
Member

@laeubi laeubi commented Oct 27, 2023

Currently PDEState.querySystemPackages leads to multiple JRTUtil.getJrtSystem calls that get cached forever.

This adds the following workaround:

  1. make sure only one analysis run in parallel
  2. clear the cache after the analysis runs

FYI @iloveeclipse

Currently PDEState.querySystemPackages leads to multiple
JRTUtil.getJrtSystem calls that get cached forever.

This adds the following workaround:
1) make sure only one analysis run in parallel
2) clear the cache after the analysis runs
@laeubi laeubi added the backport-to-tycho-4.0.x Can be added to a PR to trigger an automatic backport of the change label Oct 27, 2023
@github-actions
Copy link

Test Results

   564 files  ±0     564 suites  ±0   5h 9m 40s ⏱️ + 9m 8s
   368 tests ±0     362 ✔️ +2    6 💤 ±0  0 ±0 
1 104 runs  ±0  1 085 ✔️ +2  19 💤 ±0  0 ±0 

Results for commit e7f3cc3. ± Comparison against base commit 712efe1.

@laeubi laeubi merged commit 616d8d0 into eclipse-tycho:master Oct 28, 2023
9 checks passed
@github-actions
Copy link

💚 All backports created successfully

Status Branch Result
tycho-4.0.x

Questions ?

Please refer to the Backport tool documentation and see the Github Action logs for details

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-to-tycho-4.0.x Can be added to a PR to trigger an automatic backport of the change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant