Sure np :) On Tue, Oct 6, 2009 at 4:35 PM, Emmanuel Venisse <[email protected] > wrote:
> Thanks, I didn't see it > > Emmanuel > > On Tue, Oct 6, 2009 at 7:52 AM, <[email protected]> wrote: > > > Author: ctan > > Date: Tue Oct 6 05:52:47 2009 > > New Revision: 822134 > > > > URL: http://svn.apache.org/viewvc?rev=822134&view=rev > > Log: > > [CONTINUUM-1031] use getBuildResultsForProject instead of > > getProjectWithBuilds to improve performance when deleting a project > > > > Modified: > > > > > > continuum/branches/continuum-1.3.x/continuum-core/src/main/java/org/apache/maven/continuum/DefaultContinuum.java > > > > Modified: > > > continuum/branches/continuum-1.3.x/continuum-core/src/main/java/org/apache/maven/continuum/DefaultContinuum.java > > URL: > > > http://svn.apache.org/viewvc/continuum/branches/continuum-1.3.x/continuum-core/src/main/java/org/apache/maven/continuum/DefaultContinuum.java?rev=822134&r1=822133&r2=822134&view=diff > > > > > ============================================================================== > > --- > > > continuum/branches/continuum-1.3.x/continuum-core/src/main/java/org/apache/maven/continuum/DefaultContinuum.java > > (original) > > +++ > > > continuum/branches/continuum-1.3.x/continuum-core/src/main/java/org/apache/maven/continuum/DefaultContinuum.java > > Tue Oct 6 05:52:47 2009 > > @@ -591,7 +591,7 @@ > > { > > try > > { > > - Project project = getProjectWithBuilds( projectId ); > > + Project project = getProject( projectId ); > > > > try > > { > > @@ -660,9 +660,10 @@ > > > > log.info( "Remove project " + project.getName() + "(" + > > projectId + ")" ); > > > > - for ( Object o : project.getBuildResults() ) > > + Collection<BuildResult> buildResults = > > getBuildResultsForProject( projectId ); > > + > > + for ( BuildResult br : buildResults ) > > { > > - BuildResult br = (BuildResult) o; > > br.setBuildDefinition( null ); > > //Remove all modified dependencies to prevent SQL errors > > br.setModifiedDependencies( null ); > > > > > > >
