Bug 53271 - Remove unconditional assert in SXSSFRow.getCell(int, MissingCellPolicy)
Summary: Remove unconditional assert in SXSSFRow.getCell(int, MissingCellPolicy)
Status: RESOLVED FIXED
Alias: None
Product: POI
Classification: Unclassified
Component: SXSSF (show other bugs)
Version: 3.8-FINAL
Hardware: PC All
: P2 normal (vote)
Target Milestone: ---
Assignee: POI Developers List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-05-22 16:18 UTC by Adam Rauch
Modified: 2012-05-25 12:54 UTC (History)
0 users



Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Adam Rauch 2012-05-22 16:18:28 UTC
The first line of this method is:

    assert false;

...which makes it unusable when asserts are enabled.

Per Yegor Kozlov, "This particular assert means that SXSSFRow.getCell(int, MissingCellPolicy) is not yet covered by unit tests and *may* not be compatible with XSSF. This line will be removed in the future. Can you create a request in POI Bugzilla so that we don't forget about it?"

FWIW, I've switched to SXSSF in our application (LabKey Server) and disabled asserts, and our use of this method seems to work fine.

Workaround is to disable asserts, for example:

SXSSFRow.class.getClassLoader().setClassAssertionStatus(SXSSFRow.class.getName(), false);
Comment 1 Yegor Kozlov 2012-05-25 12:54:15 UTC
Fixed in r1342604

Yegor