Issue Type: Task Created: 2009-01-28T18:33:19.000+0000 Last Updated: 2012-11-20T21:38:11.000+0000 Status: In Progress Fix version(s): Reporter: Trevor Johns (tjohns) Assignee: None Tags: - Zend_Gdata
Related issues: Attachments: - mylyn-context.zip
Performance for Zend_Gdata is currently less than ideal. Excluding time blocked on network I/O, we're spending 650 ms to parse the YouTube top rated videos feed. This should be lower.
This is a general tracking bug. Fixes for individual performance hotspots should be filed as subtasks.
Posted by Paul Mitchell (libertus) on 2009-04-05T07:32:18.000+0000
While profiling an application that uses the Zend_Gdata_Photos API, I discovered that lookupNamespace() was particularly heavily called and dominates the runtime of the app. I tracked this down to the implementation of the various takeChildFromDOM methods.
I've attached a zip file containing a test program, test data and a patch against ZF 1.7.8 that illustrate the issue. With the patch applied I achieved a 2.5x increase in performance.
Posted by Paul Mitchell (libertus) on 2009-04-05T07:37:02.000+0000
Test program, data and patch that highlight poor performance of the various takeChildFromDOM methods.
Posted by Trevor Johns (tjohns) on 2009-04-07T17:57:10.000+0000
Paul, thanks for this patch!
However, before I can review and commit it, you'll need to have a CLA on file with the Zend Framework project. The instructions for this are here:
Would you be okay with this? If so, please let me know once your CLA has been received and processed.
Posted by Paul Mitchell (libertus) on 2009-04-08T23:57:38.000+0000
I'm happy to help!
I didn't intend for the patch to be directly applied. I produced it to illustrate the issue and help people think about a solution. I'd be delighted if you can use it to inform the development of a better patch. You surely know the Gdata architecture far better than I. :)
I'm currently investigating both this issue and ZF-5959. I hope to soon provide a more comprehensive and holistic set of recommendations to improve the performance of the Gdata library. I'm comfortable signing the CLA but it doesn't seem necessary at this early stage. If, however, you need me to sign it in order to use what I have provided thus far, of course I shall do so forthwith.
Posted by Roy Ganor (royganor) on 2010-04-14T09:08:42.000+0000
my temportary context
Have you found an issue?
See the Overview section for more details.