MartinKanters commented on a change in pull request #444: URL: https://github.com/apache/maven/pull/444#discussion_r575186110
########## File path: maven-core/src/main/java/org/apache/maven/execution/DefaultBuildResumptionAnalyzer.java ########## @@ -47,132 +41,49 @@ @Override public Optional<BuildResumptionData> determineBuildResumptionData( final MavenExecutionResult result ) { - final List<MavenProject> failedProjects = getFailedProjectsInOrder( result ); - - if ( failedProjects.isEmpty() ) + if ( !result.hasExceptions() ) { - LOGGER.info( "No failed projects found, resuming the build would not make sense." ); return Optional.empty(); } - final MavenProject resumeFromProject = failedProjects.get( 0 ); - - final String resumeFromSelector; - final List<String> projectsToSkip; - if ( isFailedProjectFirstInBuild( result, resumeFromProject ) ) - { - // As the first module in the build failed, there is no need to specify this as the resumeFrom project. - resumeFromSelector = null; - projectsToSkip = determineProjectsToSkip( result, failedProjects, 0 ); - } - else - { - resumeFromSelector = resumeFromProject.getGroupId() + ":" + resumeFromProject.getArtifactId(); - List<MavenProject> allProjects = result.getTopologicallySortedProjects(); - int resumeFromProjectIndex = allProjects.indexOf( resumeFromProject ); - projectsToSkip = determineProjectsToSkip( result, failedProjects, resumeFromProjectIndex + 1 ); - } - - boolean canBuildBeResumed = StringUtils.isNotEmpty( resumeFromSelector ) || !projectsToSkip.isEmpty(); - if ( canBuildBeResumed ) - { - return Optional.of( new BuildResumptionData( resumeFromSelector, projectsToSkip ) ); - } - else - { - return Optional.empty(); - } - } - - private boolean isFailedProjectFirstInBuild( final MavenExecutionResult result, final MavenProject failedProject ) - { - final List<MavenProject> sortedProjects = result.getTopologicallySortedProjects(); - return sortedProjects.indexOf( failedProject ) == 0; - } - - private List<MavenProject> getFailedProjectsInOrder( MavenExecutionResult result ) - { List<MavenProject> sortedProjects = result.getTopologicallySortedProjects(); - return result.getExceptions().stream() - .filter( LifecycleExecutionException.class::isInstance ) - .map( LifecycleExecutionException.class::cast ) - .map( LifecycleExecutionException::getProject ) - .filter( Objects::nonNull ) - .sorted( comparing( sortedProjects::indexOf ) ) - .collect( Collectors.toList() ); - } - - /** - * Projects after the first failed project could have succeeded by using -T or --fail-at-end. - * These projects can be skipped from later builds. - * This is not the case these projects are dependent on one of the failed projects. - * @param result The result of the Maven build. - * @param failedProjects The list of failed projects in the build. - * @param startFromProjectIndex Start looking for projects which can be skipped from a certain index. - * @return A list of projects which can be skipped in a later build. - */ - private List<String> determineProjectsToSkip( MavenExecutionResult result, - List<MavenProject> failedProjects, - int startFromProjectIndex ) - { - List<MavenProject> allProjects = result.getTopologicallySortedProjects(); - List<MavenProject> remainingProjects = allProjects.subList( startFromProjectIndex, allProjects.size() ); - - List<GroupArtifactPair> failedProjectsGAList = failedProjects.stream() - .map( GroupArtifactPair::new ) - .collect( Collectors.toList() ); - - return remainingProjects.stream() + long succeeded = sortedProjects.stream() .filter( project -> result.getBuildSummary( project ) instanceof BuildSuccess ) - .filter( project -> hasNoDependencyOnProjects( project, failedProjectsGAList ) ) - .map( project -> project.getGroupId() + ":" + project.getArtifactId() ) - .collect( Collectors.toList() ); - } - - private boolean hasNoDependencyOnProjects( MavenProject project, List<GroupArtifactPair> projectsGAs ) - { - return project.getDependencies().stream() - .map( GroupArtifactPair::new ) - .noneMatch( projectsGAs::contains ); - } - - private static class GroupArtifactPair - { - private final String groupId; - private final String artifactId; - - GroupArtifactPair( MavenProject project ) - { - this.groupId = project.getGroupId(); - this.artifactId = project.getArtifactId(); - } + .count(); - GroupArtifactPair( Dependency dependency ) + if ( succeeded == 0 ) { - this.groupId = dependency.getGroupId(); - this.artifactId = dependency.getArtifactId(); + return Optional.empty(); } - @Override - public boolean equals( Object o ) + List<MavenProject> projects = sortedProjects.stream() + .filter( project -> result.getBuildSummary( project ) == null + || result.getBuildSummary( project ) instanceof BuildFailure ) + .collect( Collectors.toList() ); + while ( true ) { - if ( this == o ) + List<MavenProject> children = projects.stream() Review comment: As far as I remember `--non-recursive` will result in `getCollectedProjects` to be empty (even if the pom has child modules). Actually --non-recursive will _only_ read the target pom (either ./pom.xml or the pom file selected via `-f pom.xml`, and none of it's child projects, I guess). So any of the selected projects will not take effect anyway. My preference would go out to listing all modules in the BuildResumptionData regardless, because: - the behavior in `DefaultGraphBuilder` could change, - users will not typically open `resume.properties`, so we do not have to optimize the flag for readability, - this `while(true)` loop adds complexity to the otherwise concise method. ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org