Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Major perf improvement with large #s of addresses and small block data #41

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

todd-richmond
Copy link

I have a customer message with 1100 addresses and tons of other small block data that takes 9.5 minutes to parse, but after optimizing both this module and OLE::Storage_lite (PR created for that project too), the time is down to < 4 seconds
There are a few fixes here

  1. eliminate linear address scan and use hash lookup instead
  2. cache the utf16 decoder as looking up every time is slow
  3. optimize methods called countless times - I had one at 4 billion calls!

lib/Email/Outlook/Message.pm Outdated Show resolved Hide resolved
@mvz mvz self-assigned this Dec 16, 2023
@todd-richmond
Copy link
Author

merge conflict resolved. My diff was based off the last release build

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants