Bug 58670 - Replace SXSSFRow's implentation of SXSSFCell[] storage with a TreeMap<Int, SXSSFCell>
Summary: Replace SXSSFRow's implentation of SXSSFCell[] storage with a TreeMap<Int, SX...
Status: RESOLVED FIXED
Alias: None
Product: POI
Classification: Unclassified
Component: SXSSF (show other bugs)
Version: 3.14-dev
Hardware: PC Linux
: P2 enhancement (vote)
Target Milestone: ---
Assignee: POI Developers List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-11-30 02:12 UTC by Javen O'Neal
Modified: 2015-11-30 07:16 UTC (History)
0 users



Attachments

Note You need to log in before you can comment on or make changes to this bug.
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.