Do the GroupBy after the ToList because it somehow messes with all th…

…ose EF Include statements and hey we can afford a few extra cycles for hash bucketing.
NuGet · Aug 14, 2013 · 5633873 · 5633873
1 parent e04cf1f
commit 5633873
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/src/NuGetGallery.Operations/Tasks/HandleQueuedPackageEditsTask.cs b/src/NuGetGallery.Operations/Tasks/HandleQueuedPackageEditsTask.cs
@@ -34,8 +34,8 @@ public override void ExecuteCommand()
                 .Include(pe => pe.Package)
                 .Include(pe => pe.Package.PackageRegistration)
                 .Include(pe => pe.Package.User)
-                .GroupBy(pe => pe.PackageKey)
-                .ToList();
+                .ToList()
+                .GroupBy(pe => pe.PackageKey);
 
             // Do edit with a 'most recent edit to this package wins - other edits are deleted' strategy.
             // Not doing editing in parallel because