Join GitHub today
GitHub is home to over 28 million developers working together to host and review code, manage projects, and build software together.Sign up
Cache XLSX workbook in Region10SpreadsheetConverter. #1675
When attempting to diagnose #1601, I noticed that an inordinate amount of time was being spent in
This improves the situation by caching the result of the call, which seems to speed up r10 bulk upload by 2-4 minutes.
It's also possible this might help with the memory issue mentioned in #1601, but I'm not sure. If we really want to reduce memory usage and speed things up, we might want to require Kelly to upload a CSV instead of an XLSX: my guess is that whatever Microsoft tool he's using on his end will have no problem doing that, and it will certainly reduce a lot of resource consumption on our end.