continuum-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From eveni...@apache.org
Subject svn commit: r387812 - in /maven/continuum/branches/continuum-1.0.x/continuum-core/src/main/java/org/apache/maven/continuum: DefaultContinuum.java buildcontroller/DefaultBuildController.java
Date Wed, 22 Mar 2006 09:39:55 GMT
Author: evenisse
Date: Wed Mar 22 01:39:54 2006
New Revision: 387812

URL: http://svn.apache.org/viewcvs?rev=387812&view=rev
Log:
Improve performance by setting the building state when the build start in DefaultBuildController
instead of setting it when we add the project in queue.

Modified:
    maven/continuum/branches/continuum-1.0.x/continuum-core/src/main/java/org/apache/maven/continuum/DefaultContinuum.java
    maven/continuum/branches/continuum-1.0.x/continuum-core/src/main/java/org/apache/maven/continuum/buildcontroller/DefaultBuildController.java

Modified: maven/continuum/branches/continuum-1.0.x/continuum-core/src/main/java/org/apache/maven/continuum/DefaultContinuum.java
URL: http://svn.apache.org/viewcvs/maven/continuum/branches/continuum-1.0.x/continuum-core/src/main/java/org/apache/maven/continuum/DefaultContinuum.java?rev=387812&r1=387811&r2=387812&view=diff
==============================================================================
--- maven/continuum/branches/continuum-1.0.x/continuum-core/src/main/java/org/apache/maven/continuum/DefaultContinuum.java
(original)
+++ maven/continuum/branches/continuum-1.0.x/continuum-core/src/main/java/org/apache/maven/continuum/DefaultContinuum.java
Wed Mar 22 01:39:54 2006
@@ -527,12 +527,6 @@
                 }
             }
 
-            project.setOldState( project.getState() );
-
-            project.setState( ContinuumProjectState.BUILDING );
-
-            store.updateProject( project );
-
             getLogger().info(
                 "Enqueuing '" + project.getName() + "' (Build definition id=" + buildDefinitionId
+ ")." );
 
@@ -1905,7 +1899,7 @@
                 }
             }
 
-            getLogger().info( " " + project.getId() + ":" + project.getName() + ":" + project.getExecutorId()
);
+            getLogger().info( " " + project.getId() + ":" + project.getName() + ":" + project.getVersion()
+ ":" + project.getExecutorId() );
         }
     }
 

Modified: maven/continuum/branches/continuum-1.0.x/continuum-core/src/main/java/org/apache/maven/continuum/buildcontroller/DefaultBuildController.java
URL: http://svn.apache.org/viewcvs/maven/continuum/branches/continuum-1.0.x/continuum-core/src/main/java/org/apache/maven/continuum/buildcontroller/DefaultBuildController.java?rev=387812&r1=387811&r2=387812&view=diff
==============================================================================
--- maven/continuum/branches/continuum-1.0.x/continuum-core/src/main/java/org/apache/maven/continuum/buildcontroller/DefaultBuildController.java
(original)
+++ maven/continuum/branches/continuum-1.0.x/continuum-core/src/main/java/org/apache/maven/continuum/buildcontroller/DefaultBuildController.java
Wed Mar 22 01:39:54 2006
@@ -130,6 +130,12 @@
 
         try
         {
+            project.setOldState( project.getState() );
+
+            project.setState( ContinuumProjectState.BUILDING );
+
+            store.updateProject( project );
+
             notifierDispatcher.buildStarted( project );
 
             Map actionContext = new HashMap();



Mime
View raw message