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

buffer file outputs #4388

Merged
merged 1 commit into from
Oct 26, 2024
Merged

buffer file outputs #4388

merged 1 commit into from
Oct 26, 2024

Conversation

Bananeweizen
Copy link
Contributor

This makes "Resolving MavenDependencyRoots..." output somewhat faster on my machine. I started from org.eclipse.tycho.core.resolver.MavenTargetDefinitionContent.generateSourceBundle(String, String, Manifest, File, IArtifactFacade), which took notable time in the profiler, and I then added buffered outputstreams and buffered writers to all FileOutputStreams.

I've run a local build of another project with this snapshot, but I have not run Tycho tests (my company machine fails too much with proxy stuff there and I can't get to my private machine for some days).

This makes "Resolving MavenDependencyRoots..." output somewhat faster on
my machine.
Copy link

Test Results

  603 files    603 suites   4h 21m 3s ⏱️
  430 tests   422 ✅  7 💤 1 ❌
1 290 runs  1 267 ✅ 22 💤 1 ❌

For more details on these failures, see this check.

Results for commit e3873fa.

Copy link
Member

@laeubi laeubi left a comment

Choose a reason for hiding this comment

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

Looks like a great improvement, too sad the OS is not capable of giving better performance by default.

@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 26, 2024
@laeubi laeubi merged commit dfab8b7 into eclipse-tycho:main Oct 26, 2024
10 of 12 checks passed
@eclipse-tycho-bot
Copy link

💚 All backports created successfully

Status Branch Result
tycho-4.0.x

Note: Successful backport PRs will be merged automatically after passing CI.

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.

3 participants