Bug 215125 - Batch service view is really slugish with medium sized site
Summary: Batch service view is really slugish with medium sized site
Status: CLOSED FIXED
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: Geclipse (show other bugs)
Version: unspecified   Edit
Hardware: PC Linux
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: kyriakos CLA
QA Contact:
URL:
Whiteboard:
Keywords: contributed
Depends on:
Blocks:
 
Reported: 2008-01-12 13:58 EST by Ariel Garcia CLA
Modified: 2014-01-09 16:14 EST (History)
2 users (show)

See Also:


Attachments
New GUI to enhance performance (73.54 KB, patch)
2008-04-16 09:22 EDT, kyriakos CLA
aog-ecl: iplog+
Details | Diff
Box bigger than necessary (78.83 KB, image/png)
2008-04-29 15:18 EDT, Ariel Garcia CLA
no flags Details
Arrows do end/start in strange places (87.02 KB, image/png)
2008-04-29 15:18 EDT, Ariel Garcia CLA
no flags Details
Layer ordering (33.42 KB, image/png)
2008-04-29 15:30 EDT, Ariel Garcia CLA
no flags Details
Solving the last issues (120.61 KB, patch)
2008-05-14 06:21 EDT, kyriakos CLA
aog-ecl: iplog+
Details | Diff
fixing the last issues (15.31 KB, patch)
2008-05-28 06:10 EDT, kyriakos CLA
aog-ecl: iplog+
Details | Diff
a small change :) (15.31 KB, patch)
2008-05-28 06:39 EDT, kyriakos CLA
aog-ecl: iplog+
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Ariel Garcia CLA 2008-01-12 13:58:05 EST
When opening the batch service view on a batch system with ~100 nodes, it takes around 5 seconds for the view to be ready. Much worse for the user is that the views in eclipse cannot be resized anymore (almost, the bar separating the views moves just a few pixels each time).
The views can be resized again if the batch system view/editor is hidden by another tab.
Comment 1 Ariel Garcia CLA 2008-01-12 14:09:56 EST
5 seconds is on my laptop, Centrino 1,8MHz and 2GBy, just for reference only, but to give an idea. Some profiling would be here really helpful to know what is taking the cicles.
Comment 2 Ariel Garcia CLA 2008-01-12 14:10:46 EST
Centrino 1,8MHz would explain the slowness, i meant 1,8GHz ;-)
Comment 3 Harald Gjermundrød CLA 2008-01-12 17:06:34 EST
It will take me some time to look at this. I have some ideas of how to "compress" the information displayed for large sites.
Comment 4 Harald Gjermundrød CLA 2008-03-27 11:24:05 EDT
Kyriacos is working on redesign of the GUI that will be applied in the next few weeks and should be ready for the M5 release.    
Comment 5 kyriakos CLA 2008-04-16 09:22:13 EDT
Created attachment 96251 [details]
New GUI to enhance performance

New GUI
Comment 6 Harald Gjermundrød CLA 2008-04-21 05:36:56 EDT
Patch is applied. Ariel would you mind testing out the new GUI and see if it is still slugish for a medium site? 
Comment 7 Ariel Garcia CLA 2008-04-29 15:16:15 EDT
No, now it works quite fast, tested with 256 nodes. Nice.

I noticed a couple of minor issues however, reporting here to avoid opening a new bug:

- the gray "box" containing all the WNs is much longer than necessary (see screenshot)

- the arrows connecting the Queues -> batch system -> WNs   behave "randomly" when moving the batch-system box around (see screenshot)
Sometimes the arrow starts/end near the middle of the border, but as you start shifting the BS box to the side the arrow ends start to "drift" towards the  middle of the box, on some WN for instance.
Comment 8 Ariel Garcia CLA 2008-04-29 15:18:01 EDT
Created attachment 98055 [details]
Box bigger than necessary
Comment 9 Ariel Garcia CLA 2008-04-29 15:18:39 EDT
Created attachment 98056 [details]
Arrows do end/start in strange places
Comment 10 Ariel Garcia CLA 2008-04-29 15:29:57 EDT
And yet another 2 small issue:

- the Batch-system box can be moved "under" the Queues or WNs boxes, and the WNs box under the queues box (see attachment, you can also see here how the arrows get crazy ;-)  Either you should forbid the boxes to "step on each other" or the BS box should be the uppermost one (then queues, then WNs below all).

- also dragging one box A over the other B, seems to interpret that we want to drag and drop it inside B:  grab box A anywhere, and move the mouse over box B.... then some "location" in box B is activated as if we could drop A into that location of B. 
(And if you actually release the mouse when the pointer is over B, then box A jumps back to its original position, doesn't get moved. This is fine for me)
Comment 11 Ariel Garcia CLA 2008-04-29 15:30:52 EDT
Created attachment 98057 [details]
Layer ordering
Comment 12 Harald Gjermundrød CLA 2008-05-05 03:26:09 EDT
Kyriakos will work on this
Comment 13 kyriakos CLA 2008-05-14 06:21:39 EDT
Created attachment 100144 [details]
Solving the last issues

Solving the last issues
Comment 14 Harald Gjermundrød CLA 2008-05-14 06:41:09 EDT
Patch applied
Comment 15 Harald Gjermundrød CLA 2008-05-16 08:52:28 EDT
Changing status to fixed and added the contributed keyword for Kyriakos patch
Comment 16 Ariel Garcia CLA 2008-05-16 15:23:30 EDT
Works much better now thanks (although there are still some tinny strange behaviours... which i do not think it is wirth caring about now ;-)

Two other size-related issues which are still important:

- i am able to resize the Queues and Nodes boxes to sizes smaller than the area required by the objects inside... If i make the box more narrow, more rows are added, but then the elements dissapear at the "bottom". And there is no visual indication of these "missing" elements. But we do not want scrollbars there, right? so perhaps a good solution is to block resizing when the area required by the boxes inside is smaller than the new "proposed" area.

- the default size of the batch-system box is too small and the text (# of WNs, # of queues) gets cut...
Comment 17 kyriakos CLA 2008-05-28 06:10:30 EDT
Created attachment 102330 [details]
fixing the last issues

fixing the last issues
Comment 18 kyriakos CLA 2008-05-28 06:39:13 EDT
Created attachment 102336 [details]
a small change :)
Comment 19 Harald Gjermundrød CLA 2008-05-28 06:44:54 EDT
Patch applied
Comment 20 Mathias Stümpert CLA 2008-08-01 11:56:32 EDT
Can this one be closed?
Comment 21 Ariel Garcia CLA 2008-08-01 12:05:07 EDT
Yes, fine now, closing
Comment 22 Ariel Garcia CLA 2009-05-03 10:21:32 EDT
Comment on attachment 100144 [details]
Solving the last issues

Applied by Harald G
Comment 23 Ariel Garcia CLA 2009-05-03 10:21:37 EDT
Comment on attachment 96251 [details]
New GUI to enhance performance

Applied by Harald G
Comment 24 Ariel Garcia CLA 2009-05-03 10:21:44 EDT
Comment on attachment 102330 [details]
fixing the last issues

Applied by Harald G
Comment 25 Ariel Garcia CLA 2009-05-03 10:21:50 EDT
Comment on attachment 102336 [details]
a small change :)

Applied by Harald G