Windows EWS went purple when it failed to build
authorap@apple.com <ap@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 30 Sep 2014 21:30:22 +0000 (21:30 +0000)
committerap@apple.com <ap@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 30 Sep 2014 21:30:22 +0000 (21:30 +0000)
https://bugs.webkit.org/show_bug.cgi?id=137267

Reviewed by Ryosuke Niwa.

* QueueStatusServer/app.yaml: Update the version.

* QueueStatusServer/model/attachment.py: (Attachment._calculate_queue_positions):
Make it work for style and win-ews queues.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@174124 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Tools/ChangeLog
Tools/QueueStatusServer/app.yaml
Tools/QueueStatusServer/model/attachment.py

index 9b12bab..0225a8e 100644 (file)
@@ -1,3 +1,15 @@
+2014-09-30  Alexey Proskuryakov  <ap@apple.com>
+
+        Windows EWS went purple when it failed to build
+        https://bugs.webkit.org/show_bug.cgi?id=137267
+
+        Reviewed by Ryosuke Niwa.
+
+        * QueueStatusServer/app.yaml: Update the version.
+
+        * QueueStatusServer/model/attachment.py: (Attachment._calculate_queue_positions):
+        Make it work for style and win-ews queues.
+
 2014-09-29  Filip Pizlo  <fpizlo@apple.com>
 
         It should be fun and easy to run every possible JavaScript benchmark from the command line
index 4c6940c..9a37934 100644 (file)
@@ -1,5 +1,5 @@
 application: webkit-queues
-version: 174115 # Bugzilla bug ID of last major change
+version: 174124 # Bugzilla bug ID of last major change
 runtime: python
 api_version: 1
 
index 2cd04e0..ec50686 100644 (file)
@@ -66,7 +66,9 @@ class Attachment(object):
         return self._cached_queue_positions
 
     def _calculate_queue_positions(self):
-        all_work_items = WorkItems.all().fetch(limit=len(Queue.all()))
+        # We don't know how many rows there are in the table (as there can be stale rows
+        # from queues that we no longer have), but it's certainly fewer that 1000.
+        all_work_items = WorkItems.all().fetch(limit=1000)
         return dict([(items.queue.name(), items.display_position_for_attachment(self.id)) for items in all_work_items if items.queue])
 
     # FIXME: This is controller/view code and does not belong in a model.