https://bz.apache.org/bugzilla/show_bug.cgi?id=57450

Stefan Thurnherr <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #33194|0                           |1
        is obsolete|                            |

--- Comment #7 from Stefan Thurnherr <[email protected]> ---
Created attachment 33207
  --> https://bz.apache.org/bugzilla/attachment.cgi?id=33207&action=edit
SXSSF Autosizing Fix

This patch introduces an AutosizeColumnTracker class for handling autosizing
when creating SXSSF files:
* monitorColumn(int) to register which columns will be autosized
* trackRow(Row) to include rows in autosize calculations
* applyAutosizeToColumn(int) to finally apply autosizing after all rows have
been added.

Additional memory overhead introduced by this class:
* Data structures to save list of monitored columns and current maxWidth for
every column
* Caching DataFormatter instance

Also I'm unsure how to handle backward compatibility, I see three
possibilities:
A. throw exception in SXSSFSheet.autosizeColumn(int)
B. keep current behaviour as-is and only add new fixed behaviour
C. remove Sheet.autosizeColumn() and migrate all Sheet implementations to this
new autosizing implementation

I'm willing to improve the patch if required, so hopefully it will be included
in the next Apache POI release :-)

-- 
You are receiving this mail because:
You are the assignee for the bug.

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to