tel8618217223380 / sagetv-addons

Automatically exported from code.google.com/p/sagetv-addons
0 stars 0 forks source link

Add unique queue id to list of client's active tasks #240

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
Please add the QueueId to the table.  If a user has multiple tasks of the same 
type running (as shown in the screen shot) and needs to kill one they have no 
idea which one they're killing.

Original issue reported on code.google.com by de...@battams.ca on 13 Nov 2010 at 8:09

Attachments:

GoogleCodeExporter commented 9 years ago
Fixed in next build.

Original comment by tom.mira...@gmail.com on 13 Nov 2010 at 9:11

GoogleCodeExporter commented 9 years ago
The queue id addition is fine, but for some reason the list of tasks for a 
client does not stay in sync.  It always seems to keep the last one around even 
though it shouldn't.

You'll notice that 1135 is still shown as WAITING even though the queue has no 
client assigned to task 1135.  Then after 1135 completes, the UI still shows it 
running and it never goes away even though the task has already failed (as 
shown in capture 34).

Original comment by de...@battams.ca on 14 Nov 2010 at 3:05

Attachments:

GoogleCodeExporter commented 9 years ago
I assume you are closing the dialog and reopening it again.  How long are you 
waiting before reopening it?  The TaskList should get updated every 2-3 seconds 
so if you close the dialog and quickly reopen it, it will not get updated.

I think I can actually update the dialog while it it open, I'll try to get that 
working for the next build.

Original comment by tom.mira...@gmail.com on 14 Nov 2010 at 4:25

GoogleCodeExporter commented 9 years ago
For the case where that last task hangs around, it was at least 30 seconds 
later that I reloaded that panel and the task count was still showing as 1 even 
though there were no active tasks assigned to that client for at least the 30 
seconds (probably longer).

Original comment by de...@battams.ca on 14 Nov 2010 at 4:32

GoogleCodeExporter commented 9 years ago
I think I can chalk this one up to stupidity on my part.  There is some test 
code in there I forgot to take out.  Can you confirm by searching the STV for 
the "FIXME" that has a child action widget that contains QTL = DataUnion(QTL, 
ThisTask).  Deleting the action widget will probably cure the proplem.  Can you 
try it out and let me know if that works?

Original comment by tom.mira...@gmail.com on 14 Nov 2010 at 5:24

GoogleCodeExporter commented 9 years ago
The suggested fix seems to have fixed the issue.

Original comment by de...@battams.ca on 14 Nov 2010 at 7:40

GoogleCodeExporter commented 9 years ago
OK.  I'll make sure the test code is no in the next build.

Original comment by tom.mira...@gmail.com on 14 Nov 2010 at 8:26

GoogleCodeExporter commented 9 years ago

Original comment by de...@battams.ca on 9 Apr 2013 at 2:12