You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@poi.apache.org by "Andrew C. Oliver" <ac...@apache.org> on 2003/02/04 23:10:54 UTC

Re: ExtSSTRecord [Scanned For Viruses] [Scanned For Viruses] [Sca nned For Viruses] [Scanned For Viruses]

Check out the performance branch...  Do you think you could impelment it 
there too?  (It will be radically different as no treemap)...

-Andy

Height, Jason wrote:

>The performance cost is not bad, I just wanted to mention that there is one.
>
>
>What happens in my implementation is that I need to be able to write out a
>set of rows and a set of cells for those rows. Since a TreeMap is used to
>store the rows and values I call TreeMap.values().iterator() to traverse the
>rows and values.
>
>So the cost is the memory required to create a Collection, Iterator and then
>the performance cost of iterating the rows & values. In my experiments the
>cost isn't noticeable, although I was only using excel files that have a
>small number of rows (~35), but around 400 cells.
>
>Jason
>
>
>-----Original Message-----
>From: Andrew C. Oliver [mailto:acoliver@apache.org] 
>Sent: Tuesday, 4 February 2003 23:55
>To: POI Developers List
>Subject: Re: ExtSSTRecord [Scanned For Viruses] [Scanned For Viruses] [Sca
>nned For Viruses] [Scanned For Viruses]
>
>  
>
>>One point I made in the bug report was that there is a performance cost
>>associated with correctly implementing these records (need to temporary
>>Collections and iterators over the TreeMap collections in the Aggregate
>>classes). I thought that I would address the performance issues of the
>>    
>>
>patch
>  
>
>>after the *working* patch is applied.
>>
>>
>>    
>>
>if its not *toooooo* bad then +1.. . If you mean like years then well I 
>might feel differently.
>
>  
>
>>Jason
>>
>>-----Original Message-----
>>From: Andrew C. Oliver [mailto:acoliver@apache.org] 
>>Sent: Tuesday, 4 February 2003 13:53
>>To: POI Developers List
>>Subject: Re: ExtSSTRecord [Scanned For Viruses] [Scanned For Viruses]
>>[Scanned For Viruses]
>>
>>Awesome!
>>
>><snip>
>>
>>---------------------------------------------------------------------------
>>    
>>
>-----------------------------------------
>  
>
>>This e-mail (including attachments) is confidential information of
>>    
>>
>Australian Submarine Corporation Pty Limited (ASC).  It may also be legally
>privileged.  Unauthorised use and disclosure is prohibited.  ASC is not
>taken to have waived confidentiality or privilege if this e-mail was sent to
>you in error. If you have received it in error, please notify the sender
>promptly.  While ASC takes steps to identify and eliminate viruses, it
>cannot confirm that this e-mail is free from them.  You should scan this
>e-mail for viruses before it is used.  The statements in this e-mail are
>those of the sender only, unless specifically stated to be those of ASC by
>someone with authority to do so.
>  
>
>> 
>>
>>    
>>
>
>
>
>---------------------------------------------------------------------
>To unsubscribe, e-mail: poi-dev-unsubscribe@jakarta.apache.org
>For additional commands, e-mail: poi-dev-help@jakarta.apache.org
>
>--------------------------------------------------------------------------------------------------------------------
>This e-mail (including attachments) is confidential information of Australian Submarine Corporation Pty Limited (ASC).  It may also be legally privileged.  Unauthorised use and disclosure is prohibited.  ASC is not taken to have waived confidentiality or privilege if this e-mail was sent to you in error. If you have received it in error, please notify the sender promptly.  While ASC takes steps to identify and eliminate viruses, it cannot confirm that this e-mail is free from them.  You should scan this e-mail for viruses before it is used.  The statements in this e-mail are those of the sender only, unless specifically stated to be those of ASC by someone with authority to do so.
>
>  
>