Bug 176574 - OutOfMemory Exception during th refresh for Eclipse's Bugzilla
Summary: OutOfMemory Exception during th refresh for Eclipse's Bugzilla
Keywords:
Status: CLOSED UPSTREAM
Alias: None
Product: Fedora
Classification: Fedora
Component: eclipse-bugzilla
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Igor Foox
QA Contact:
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2005-12-26 21:22 UTC by Stephan Michels
Modified: 2007-11-30 22:11 UTC (History)
2 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2006-01-06 22:29:01 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)
Patch against org.eclipse.team.bugs (CVS HEAD) (987 bytes, patch)
2005-12-26 21:23 UTC, Stephan Michels
no flags Details | Diff
Patch against org.eclipse.team.bugzilla (CVS HEAD) (12.15 KB, patch)
2005-12-26 21:24 UTC, Stephan Michels
no flags Details | Diff
Revised patch (75.46 KB, patch)
2006-01-05 22:12 UTC, Igor Foox
no flags Details | Diff
Revised patch2 (84.02 KB, patch)
2006-01-06 22:25 UTC, Igor Foox
no flags Details | Diff
Cleaned up patch (10.89 KB, patch)
2006-01-09 16:17 UTC, Igor Foox
no flags Details | Diff

Description Stephan Michels 2005-12-26 21:22:12 UTC
From Bugzilla Helper:
User-Agent: Mozilla/5.0 (X11; U; Linux ppc; en-US; rv:1.8) Gecko/20051202 Ubuntu/1.4.99+1.5rc3.dfsg-1ubuntu4 Epiphany/1.9.3.1 Firefox/1.5

Description of problem:
Hi,
the plugin doesn't work with Eclipse's Bugzilla. I got a OutOfMemoryException
during the refresh. I created a patch against the CVS HEAD to fix various
problems I got.

Problems, I solved:
1. OOME during the parsing of Eclipse's Bugzilla query page.
2. NPE if I deselect all tree items.
3. Error in the creation of the status object for exceptions.
4. Read content type for attachments, before I got problems because the id 
   of the attachment wasn't read correct.
5. Make borders around lists.
6. UTF8(whatever)Exception because a java.io.Reader was used to read the binary   
   content.
7. Use bugzillaAttachment.getDataAsString() only for text/* types.
8. Add Ubuntu's bugzilla



Version-Release number of selected component (if applicable):


How reproducible:
Always

Steps to Reproduce:
1.
2.
3.
  

Additional info:

Comment 1 Stephan Michels 2005-12-26 21:23:44 UTC
Created attachment 122587 [details]
Patch against org.eclipse.team.bugs (CVS HEAD)

Comment 2 Stephan Michels 2005-12-26 21:24:46 UTC
Created attachment 122588 [details]
Patch against org.eclipse.team.bugzilla (CVS HEAD)

Comment 3 Igor Foox 2006-01-05 22:12:13 UTC
Created attachment 122845 [details]
Revised patch

Hi Stephan,

Thanks for such an extensive patch to the plugin :). I've tested your patch and
it seems to work well. I've made some small syntactic changes, such as leaving
the explicit imports instead of .*, as well as making the instantiation of the
Status object in BugzillaPlugin a bit more readable. 

If nothing else comes up I'll commit this.

Comment 4 Igor Foox 2006-01-06 22:25:55 UTC
Created attachment 122896 [details]
Revised patch2

I am committing the patch, with some other minor changes, this is the final
form.

Stephan, thanks again for the work.

Comment 5 Andrew Overholt 2006-01-07 15:48:30 UTC
The patch that you attached is a little messed up, Igor.  Lots of the files are
completely changed according to the patch ... maybe you re-indented?  It'd be
cool if the contextual differences were more apparent.  Thanks.

Comment 6 Igor Foox 2006-01-09 15:17:50 UTC
Opening the patch up in vi shows that it's most likely that the files that were
changed without real changes had windows line endings. Since I already comitted
to CVS and I don't see converting win line endings to unix line endings as a big
transgression, I'll leave it in CVS as is, but I'll repost a patch without these
files for easier reading. :)

Igor

Comment 7 Igor Foox 2006-01-09 16:17:16 UTC
Created attachment 122953 [details]
Cleaned up patch

Here's a patch without the line-ending changes.


Note You need to log in before you can comment on or make changes to this bug.