Bug #85370 for iCal-Parser: Speeding up iCal::Parser

Thu May 16 23:21:50 2013 bruce [...] momjian.us - Ticket created

Subject:	Speeding up iCal::Parser
Date:	Thu, 16 May 2013 23:21:31 -0400
To:	bug-iCal-Parser [...] rt.cpan.org
From:	Bruce Momjian <bruce [...] momjian.us>

parse_strings() on a 360k Google-produced ical string took 35 seconds on my server-grade hardware, which seemed slow. I found if I set 'tz' to equal my local timezone, the duration decreased to 14 seconds. It might be nice if you could cache the current timezone and avoid lookups for cases where 'tz' is not set. Attached is my inclusive and exclusive Perl profiling output, in case you see other optimizations to try. -- Bruce Momjian <bruce@momjian.us> http://momjian.us EnterpriseDB http://enterprisedb.com

Message body is not shown because sender requested not to inline it.

Thu May 16 23:32:49 2013 https://www.google.com/accounts/o8/id?id=AItOawlsFr716ITHX5fFqEoy1RDF6vBODDmICWg - Correspondence added

From:

bruce [...] momjian.us

I should have mentioned my iCal::Parser version is 1.16, and my Perl version is v5.10.1 on Debian 6. Server is an 8-core with 24GB of RAM.

Thu May 16 23:32:49 2013 The RT System itself - Status changed from 'new' to 'open'

Sat Jun 15 08:54:46 2013 RFRANKEL [...] cpan.org - Correspondence added

$self->{tz} now acceptis/set to a DateTime::TimeZone

Sat Jun 15 08:54:47 2013 RFRANKEL [...] cpan.org - Status changed from 'open' to 'resolved'

Sat Jun 15 08:55:32 2013 RFRANKEL [...] cpan.org - Correspondence added

Sorry, forgot: fixed in version 1.20

Sat Jun 15 13:05:10 2013 https://www.google.com/accounts/o8/id?id=AItOawlsFr716ITHX5fFqEoy1RDF6vBODDmICWg - Correspondence added

On Sat Jun 15 08:55:32 2013, RFRANKEL wrote: Show quoted text

> Sorry, forgot: fixed in version 1.20

I can confirm that 1.20 fixes the problem with poor performance if tz is not set. Thank you.