Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 50684 - Lock not available but no jobs are running
Summary: Lock not available but no jobs are running
Status: CLOSED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 3.0   Edit
Hardware: PC Windows 2000
: P2 major (vote)
Target Milestone: ---   Edit
Assignee: Tod Creasey CLA
QA Contact:
URL:
Whiteboard:
Keywords:
: 50757 (view as bug list)
Depends on:
Blocks:
 
Reported: 2004-01-27 11:00 EST by John Arthorne CLA
Modified: 2016-03-07 01:38 EST (History)
3 users (show)

See Also:


Attachments
Stack trace (31.29 KB, text/plain)
2004-01-27 11:01 EST, John Arthorne CLA
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description John Arthorne CLA 2004-01-27 11:00:38 EST
I20040121

What I did:

1) Initiated a background project tag
2) While waiting, I tried typing in an editor in a different project

-> Busy "blocked by background job" dialog popped up.

3) Hit cancel in the dialog.
4) Try to edit again
5) Repeat 3-4 a few times

-> Finally the background tag is done, but it still claims to be blocked. Will
attach the stack.

It looks like the ProgressMonitorJobsDialog is not sending the cancel through to
the underlying runnable, but just closing immediately and leaving behind the
waiting task.  I need to investigate a bit more.
Comment 1 John Arthorne CLA 2004-01-27 11:01:36 EST
Created attachment 7589 [details]
Stack trace
Comment 2 Tod Creasey CLA 2004-01-27 11:13:18 EST
John we have updated this dialog in the latest integration build so cancel of 
the dialog is no longer possible (although you can cancel the jobs). I think 
this is obsolete.
Comment 3 John Arthorne CLA 2004-01-27 11:42:04 EST
What if the user really does want to cancel the foreground activity?  They
realize that they have tried to do something that conflicts with the background
activity, so they want to back out and do something else instead while they wait
for the background job to complete.

We support cancelation when blocked, so I don't see a technical reason why we
can't do it.  We just have to get the BlockedJobsDialog cancel button to call
setCanceled on the EventLoopProgressMonitor. The main thread, which is stuck
waiting for a scheduling rule, checks for cancelation on the monitor while it is
waiting.
Comment 4 Tod Creasey CLA 2004-01-27 12:03:58 EST
The new dialog doesn't allow cancel as it is only to show blocking - i.e. 
there are too cases now - one where you are running an operation and one where 
you are reporting a block only.
Comment 5 John Arthorne CLA 2004-01-27 14:21:46 EST
I agree with having two different dialogs for these two cases, but I'm saying
the "blocked" case should still support cancelation.  We didn't do this
correctly before (hence this bug), but it can be done.  I think the "blocked"
dialog needs two buttons:

"Cancel selected operations" -> Cancel background jobs
"Try later" -> Cancel the foreground operation.

Note: I can reproduce the problem in I20040127 by closing the "blocked" dialog
using the shell close button or the close action in the shell's drop down menu.

Moving to UI, but I can investigate a patch if you like.
Comment 6 Jean-Michel Lemieux CLA 2004-01-28 11:02:19 EST
The user still has to be able to cancel the modal operation when blocked. We 
want to demo this next week at EclipseCon.
Comment 7 Tod Creasey CLA 2004-01-28 15:14:08 EST
Suggestions implemented and released into HEAD. I will close this Bug if all 
of your needs have been satisified.
Comment 8 John Arthorne CLA 2004-01-28 15:55:33 EST
I'm never satisfied :)
Comment 9 Tod Creasey CLA 2004-01-28 15:57:51 EST
Closing pending comments.
Comment 10 John Arthorne CLA 2004-01-28 17:07:23 EST
*** Bug 50757 has been marked as a duplicate of this bug. ***
Comment 11 Philipe Mulet CLA 2004-01-30 11:35:08 EST
I don't see any improvement in build 200401290841

Comment 12 John Arthorne CLA 2004-01-30 11:42:38 EST
The UI changes were not released in that build.
Comment 13 Tod Creasey CLA 2004-01-30 12:57:40 EST
UI did not resubmit until 20040130
Comment 14 Tod Creasey CLA 2005-05-10 14:55:54 EDT
Marking closed