apache / apache/maven-verifier

[MSHARED-1074] Improvement performance of getMavenVersion in ForkedLauncher

Open
#162 2 comments 0 reactions 0 assignees View on GitHub
bug
Dominant language
Java
Stars
6
Forks
17
Avg merge
12m
Merged PRs (30d)
1

Description

**[Slawomir Jaranowski](https://issues.apache.org/jira/secure/ViewProfile.jspa?name=slawekjaranowski)** opened **[MSHARED-1074](https://issues.apache.org/jira/browse/MSHARED-1074?redirect=false)** and commented

In method `getMavenVersion` in ForkedLauncher new Maven process is forked every time.

We can cache output by Maven home or try to read from properties files in Maven home `/lib/maven-core-xxx.jar!/META-INF/maven/org.apache.maven/maven-core/pom.properties`

There is also possibility to use Maven wrapper for executing - in this case we can read version form maven-wrapper.properties

---

**Issue Links:**
- [MSHARED-1140](https://issues.apache.org/jira/browse/MSHARED-1140) Simplify Maven version extraction in the forked launcher

Contributor guide

No contributing guide indexed for this repository

Research direction

Start in ForkedLauncher at getMavenVersion and review how the Maven process is forked on each call. Compare the issue's proposed Maven-home cache, maven-core pom.properties path, and maven-wrapper.properties approach, then check the linked MSHARED-1140 issue for the intended direction; done means Maven version extraction avoids unnecessary process launches without breaking wrapper handling.

Written by the indexing model from the issue text.

Assessment

Tech stack
java
Domain
build-system
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.