Bug 180917 - [plan] Job viewer for showing all job information
Summary: [plan] Job viewer for showing all job information
Status: CLOSED FIXED
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: Geclipse (show other bugs)
Version: unspecified   Edit
Hardware: All All
: P3 enhancement (vote)
Target Milestone: ---   Edit
Assignee: Mariusz Wojtysiak CLA
QA Contact:
URL:
Whiteboard:
Keywords: contributed
Depends on:
Blocks:
 
Reported: 2007-04-04 06:25 EDT by Pawel Wolniewicz CLA
Modified: 2014-01-09 16:18 EST (History)
1 user (show)

See Also:


Attachments
Patch containing JobDetails view implemented as extension point (10.06 KB, patch)
2007-07-31 04:07 EDT, Mariusz Wojtysiak CLA
no flags Details | Diff
Patch with later changes in the same plugin. First patch can be ignored, because this patch also contains changes from first patch. (115.86 KB, patch)
2007-08-07 09:32 EDT, Mariusz Wojtysiak CLA
no flags Details | Diff
Patch with renamed package: JobDetailsNew to JobDetails (63.62 KB, patch)
2007-08-09 03:32 EDT, Mariusz Wojtysiak CLA
aog-ecl: iplog+
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Pawel Wolniewicz CLA 2007-04-04 06:25:56 EDT
 
Comment 1 Mariusz Wojtysiak CLA 2007-04-30 06:59:30 EDT
For now  JobDetails show only data from IGridJob and IGridJobDescription.
It contains 2 sections: General and Description

I have to develop sections for JsdlJobDescription.
Comment 2 Mariusz Wojtysiak CLA 2007-07-31 04:07:52 EDT
Created attachment 75006 [details]
Patch containing JobDetails view implemented as extension point

I refactored code for JobDetails view in following way:
- package eu.geclipse.ui.views.jobdetails was removed
- new package eu.geclipse.ui.views.jobdetailsNEW was added

Unfortunatelly I cannot rename jobdetailsNEW do jobdetails, till old package is completely removed from repository. 
So for now please:
- just apply patch despite of wrong plackage name: jobdetailsNEW
- reassign bug to me
- in next patch I will rename package to jobdetails

Now job details view is developed as extension point. So it's possible to show different sections/details for different middlewares. (gLite details are developed in glite plugin, and I will commit it to CVS, when this patch will be commited).
Comment 3 Mathias Stümpert CLA 2007-08-02 09:00:59 EDT
Reassigning to Markus since I will be on holiday for the next 2 weeks
Comment 4 Mariusz Wojtysiak CLA 2007-08-07 09:32:21 EDT
Created attachment 75538 [details]
Patch with later changes in the same plugin. First patch can be ignored, because this patch also contains changes from first patch.
Comment 5 Markus Knauer CLA 2007-08-07 10:33:50 EDT
 (In reply to comment #2)
> Unfortunatelly I cannot rename jobdetailsNEW do jobdetails, till old package is
> completely removed from repository.
> So for now please:
> - just apply patch despite of wrong plackage name: jobdetailsNEW
> - reassign bug to me
> - in next patch I will rename package to jobdetails

Mariusz, I don't like the idea of having the repository (again) in a state with compile and dependency errors, but I see the refactoring problems with Subversion here. I would suggest that we do the changes together and minimise the duration of a non-compiling state.
Comment 6 Mariusz Wojtysiak CLA 2007-08-09 03:32:19 EDT
Created attachment 75730 [details]
Patch with renamed package: JobDetailsNew to JobDetails
Comment 7 Katarzyna Bylec CLA 2007-08-09 04:12:04 EDT
Together with Mariusz we fixed all errors and successfully applied the patch.
Comment 8 Markus Knauer CLA 2007-08-10 07:35:37 EDT
Well, that's even better. Thanks.
Don't forget to set the status to 'fixed' if it is really fixed...
Comment 9 Mariusz Wojtysiak CLA 2007-08-10 07:40:45 EDT
Job details view developed for g-lite.
Comment 10 Ariel Garcia CLA 2007-10-25 08:21:22 EDT
Closing this one
Comment 11 Ariel Garcia CLA 2009-05-03 08:57:23 EDT
Comment on attachment 75730 [details]
Patch with renamed package: JobDetailsNew to JobDetails

Applied by Katarzyna