Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 338549

Summary: [patch] mark multiple tasks as read
Product: z_Archived Reporter: Thomas Ehrnhoefer <thomas.ehrnhoefer>
Component: MylynAssignee: Project Inbox <mylyn-triaged>
Status: CLOSED MOVED QA Contact:
Severity: enhancement    
Priority: P3 CC: steffen.pingel
Version: unspecified   
Target Milestone: ---   
Hardware: PC   
OS: Windows 7   
Whiteboard:
Bug Depends on: 271905    
Bug Blocks:    
Attachments:
Description Flags
patch v1 none

Description Thomas Ehrnhoefer CLA 2011-03-01 10:54:05 EST
Currently the setTaskRead method in the TaskDataManager invokes a tasklist runnable.
I am using a few handlers to mark multiple tasks (matching certain tests) as read. If there is a lot of results, and I am bootstrapped, I am running into out of memory issues and a JVM crash.

Not sure if it would be the right approach, but a @setTaskRead(final ITask[] itasks, final boolean read) {@ method would seem to solve my problem.
Comment 1 Thomas Ehrnhoefer CLA 2011-03-01 10:55:51 EST
Created attachment 190044 [details]
patch v1

this would be my suggestion. it worked for me, although I saw many "Mark Read" jobs in my progress, and during that the UI seems to be not fully responsive, hard to tell though.
Comment 2 Steffen Pingel CLA 2011-04-13 16:14:39 EDT
The patch looks good and I think it's the right approach to aggregate task list notifications. This should also help with bug 271905. I'll apply if you can you add a test case?
Comment 3 Thomas Ehrnhoefer CLA 2011-04-18 17:04:47 EDT
Not sure when I will have time for this, as it looks right now, most likely not for Indigo, sorry
Comment 4 Eclipse Webmaster CLA 2022-11-15 11:45:08 EST
Mylyn has been restructured, and our issue tracking has moved to GitHub [1].

We are closing ~14K Bugzilla issues to give the new team a fresh start. If you feel that this issue is still relevant, please create a new one on GitHub.

[1] https://github.com/orgs/eclipse-mylyn