Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Parallel processing of files #568

Merged
merged 1 commit into from
May 26, 2024
Merged

Conversation

mkarg
Copy link
Contributor

@mkarg mkarg commented May 23, 2024

Projects consisting of possibly thousands of files can take long time to get processed, mostly because file operations are rather slow (compared to in-memory operations), so it makes sense to speed up things if possible without breaking backwards compatibility.

A potential solution is to use several threads, so while the first thread is still blocked by an I/O operation, another thread can start processing the next file already. With multi-core CPUs being ubiquitous these days, such a rather small change can improve the build time considerably, in particular for huge projects with thousands of files and / or slows disks (like typically on NTFS). For single-core CPUs the drawback is neglectable.

NB: This PR mimics the solution of mojohaus/build-helper-maven-plugin#201

@mkarg mkarg marked this pull request as draft May 23, 2024 14:25
@mkarg mkarg force-pushed the parallel-files branch from 56fa4cd to b37353d Compare May 24, 2024 07:22
@mkarg mkarg force-pushed the parallel-files branch from b37353d to 751b268 Compare May 24, 2024 13:10
@mkarg mkarg marked this pull request as ready for review May 26, 2024 08:18
@mkarg
Copy link
Contributor Author

mkarg commented May 26, 2024

@slachiewicz @slawekjaranowski @khmarbaise Kindly requesting your review. :-)

@slawekjaranowski slawekjaranowski merged commit 30552a0 into mojohaus:master May 26, 2024
20 checks passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants