[ 
https://issues.apache.org/jira/browse/MNG-7629?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17649082#comment-17649082
 ] 

ASF GitHub Bot commented on MNG-7629:
-------------------------------------

gnodet commented on code in PR #912:
URL: https://github.com/apache/maven/pull/912#discussion_r1051658332


##########
maven-core/src/main/java/org/apache/maven/ReactorReader.java:
##########
@@ -64,28 +63,21 @@
 class ReactorReader implements MavenWorkspaceReader {
     public static final String HINT = "reactor";
 
-    private static final Collection<String> COMPILE_PHASE_TYPES =
-            Arrays.asList("jar", "ejb-client", "war", "rar", "ejb3", "par", 
"sar", "wsr", "har", "app-client");
-
     private static final Logger LOGGER = 
LoggerFactory.getLogger(ReactorReader.class);
 
     private final MavenSession session;
     private final Map<String, MavenProject> projectsByGAV;
     private final Map<String, List<MavenProject>> projectsByGA;
     private final WorkspaceRepository repository;
 
-    private Function<MavenProject, String> projectIntoKey =
-            s -> ArtifactUtils.key(s.getGroupId(), s.getArtifactId(), 
s.getVersion());
-
-    private Function<MavenProject, String> projectIntoVersionlessKey =
-            s -> ArtifactUtils.versionlessKey(s.getGroupId(), 
s.getArtifactId());
-
     @Inject
     ReactorReader(MavenSession session) {
         this.session = session;
-        this.projectsByGAV = 
session.getAllProjects().stream().collect(toMap(projectIntoKey, identity()));
+        this.projectsByGAV = session.getAllProjects().stream()

Review Comment:
   It's not.  I'll revert this hunk to minimize the changes.





> Revert MNG-6118 and provide a better solution for it
> ----------------------------------------------------
>
>                 Key: MNG-7629
>                 URL: https://issues.apache.org/jira/browse/MNG-7629
>             Project: Maven
>          Issue Type: Task
>            Reporter: Guillaume Nodet
>            Assignee: Guillaume Nodet
>            Priority: Major
>             Fix For: 4.0.0-alpha-4
>
>




--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to