Tweaking UpdateStatisticsJob to update PackageRegistrations condition…

…ally
NuGet · Aug 23, 2012 · e65e331 · e65e331
1 parent cfb2db1
commit e65e331
Showing 1 changed file with 14 additions and 11 deletions.
diff --git a/Website/Infrastructure/Jobs/UpdateStatisticsJob.cs b/Website/Infrastructure/Jobs/UpdateStatisticsJob.cs
@@ -66,20 +66,23 @@ UPDATE p
     FROM Packages p INNER JOIN @DownloadStats stats
     ON p.[Key] = stats.PackageKey
 
+    IF @@ROWCOUNT > 0
+    BEGIN
+        UPDATE pr
+        SET pr.DownLoadCount = totals.DownloadCount
+        FROM PackageRegistrations pr INNER JOIN
+        (
+            SELECT PackageRegistrationKey, DownloadCount = SUM(DownloadCount)
+            FROM Packages
+            GROUP BY PackageRegistrationKey
+        ) as totals
+        ON pr.[Key] = totals.PackageRegistrationKey
+    END    
+
     UPDATE GallerySettings
     SET DownloadStatsLastAggregatedId = @mostRecentStatisticsId
 
-COMMIT TRANSACTION
-
-UPDATE pr
-SET pr.DownLoadCount = totals.DownloadCount
-FROM PackageRegistrations pr INNER JOIN
-(
-    SELECT PackageRegistrationKey, DownloadCount = SUM(DownloadCount)
-    FROM Packages
-    GROUP BY PackageRegistrationKey
-) as totals
-ON pr.[Key] = totals.PackageRegistrationKey";
+COMMIT TRANSACTION";
             using (var context = _contextThunk())
             {
                 context.Database.ExecuteSqlCommand(sql);