search

monitorjbl / excel-streaming-reader

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

start process of removing ctrst class #149

Closed pjfanning closed 6 years ago

pjfanning commented 6 years ago

POI team are trying to remove dependence on the XMLBeans generated class like CTRst. This change reduces the dependence on this class. I have a follow up for when POI 4.0.0 is released shortly that will remove the need for that class at all. (https://github.com/pjfanning/excel-streaming-reader/tree/poi-4 and https://github.com/pjfanning/poi-shared-strings)

Relates to https://github.com/monitorjbl/excel-streaming-reader/issues/125

monitorjbl commented 6 years ago

Thanks, I'd like to play around with this locally before another release goes out. I'm surprised this works, but it certainly looks a lot more efficient than having to go through JSON serde every time.