Bug 58670

Summary: Replace SXSSFRow's implentation of SXSSFCell[] storage with a TreeMap<Int, SXSSFCell>
Product: POI Reporter: Javen O'Neal <onealj>
Component: SXSSFAssignee: POI Developers List <dev>
Status: RESOLVED FIXED    
Severity: enhancement    
Priority: P2    
Version: 3.14-dev   
Target Milestone: ---   
Hardware: PC   
OS: Linux   

Description Javen O'Neal 2015-11-30 02:12:28 UTC
From dev mailing list [1]
Replace SXSSFRow's data structure for storing SXSSFCells with a TreeMap, which uses less memory than an Array for sparse spreadsheets, reduces the number of lines of code, and is faster for some methods (trivially slower for others). Since the purpose of the streaming workbook API is to have a lower memory footprint and not speed, this seems like a good tradeoff.

[1] https://mail-archives.apache.org/mod_mbox/poi-dev/201511.mbox/%3CCABdJj57-Z1sMtdTXdTj8-r_PJ2TmRFHJ%3DM2RfHUU49hiATgW7Q%40mail.gmail.com%3E
Comment 1 Javen O'Neal 2015-11-30 07:12:16 UTC
Added in r1717192 and r1717195. No need to update changes.html since this is internal to POI.
Comment 2 Javen O'Neal 2015-11-30 07:16:28 UTC
On second thought, this change might affect the memory consumption or speed of SXSSFWorkbooks, which users should know about in case they notice an increase or decrease in memory or speed.

Update change log in r1717196.