search

monitorjbl / excel-streaming-reader

An easy-to-use implementation of a streaming Excel reader using Apache POI
Apache License 2.0
946 stars 343 forks source link

Fix npe empty cells #175

Closed shlomiken closed 5 years ago

shlomiken commented 5 years ago

We have tried to read a very large file 500K records , with lots of columns. so we used the SST cache not to overload memory with the shared string table.

some columns have empty values. this caused a NPE cause the write to sst cache file was trying to read the bytes length from the string , and was not protection of null

monitorjbl commented 5 years ago

Looks good to me!

shlomiken commented 5 years ago

hi , how deploy/build/delivery works , will this be in 2.0.0 or a coming version thanks Shlomi

fabien-chebel commented 5 years ago

@monitorjbl Hello! I ran into this "empty cell" issue as well after upgrading to v2.1.0.

Do you plan on releasing a new version of your library soon?

Bzil commented 3 years ago

Hey @monitorjbl do you still maintain this package? If yes, is any release plan in the future?