Project

General

Profile

Actions

Bug #1219

closed

queue time estimate takes too long if the queue is large

Added by Anchi Cheng over 13 years ago. Updated over 13 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Amber Herold
Category:
myamiweb
Start date:
03/03/2011
Due date:
% Done:

0%

Estimated time:
Affected Version:
Appion/Leginon 2.1.1
Show in known bugs:
No
Workaround:

Description

It becomes much slower that loi.php refresh time and causes stall and slow down.


Related issues 1 (0 open1 closed)

Related to Leginon - Bug #1218: queue time estimation does not include focusing timeClosedErica Jacovetty03/03/2011

Actions
Actions #1

Updated by Anchi Cheng over 13 years ago

  • Status changed from Assigned to In Code Review
  • Assignee changed from Anchi Cheng to Eric Hou

r15476 fixes both this and #1218

Note that getqcount.php is called from loi.php and refreshed each time loi refreshes.

For this part, a link to queuecount.php is displayed rather than running the long query if there are more than 500 targets in queue of the session. Also, a limit is put on the query for estimating the average processing time so that if a small unprocessed queue is in a session of a large number of processed queue will be faster and more accurate because the estimate will be based on later ones.

Eric, if you know how to speed this series of queries up, please let me know.

Testing:
1. visit myamiweb/loi.php for session 11feb28a. This has a big queue, but by the time this code is tested, it will be done so that the query should take a few seconds at most at its linked page.

2. visit the same site on a session that is currently running. If the queue is not big, the box showing the result should refresh within seconds.

Actions #2

Updated by Eric Hou over 13 years ago

  • Status changed from In Code Review to In Test

see comment in issue 1218

Actions #3

Updated by Eric Hou over 13 years ago

  • Assignee changed from Eric Hou to Amber Herold
Actions #4

Updated by Amber Herold over 13 years ago

  • Status changed from In Test to Closed
Actions

Also available in: Atom PDF