Zend Framework

Reduce Zend_Gdata latency

Details

  • Type: Task Task
  • Status: In Progress In Progress
  • Priority: Major Major
  • Resolution: Unresolved
  • Affects Version/s: 1.7.3
  • Fix Version/s: None
  • Component/s: Zend_Gdata
  • Labels:
    None

Description

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.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Paul Mitchell added a comment -

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.

Show
Paul Mitchell added a comment - 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.
Hide
Permalink
Paul Mitchell added a comment -

Test program, data and patch that highlight poor performance of the various takeChildFromDOM methods.

Show
Paul Mitchell added a comment - Test program, data and patch that highlight poor performance of the various takeChildFromDOM methods.
Hide
Permalink
Trevor Johns added a comment -

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:

http://framework.zend.com/community/contribute

Would you be okay with this? If so, please let me know once your CLA has been received and processed.

Show
Trevor Johns added a comment - 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: http://framework.zend.com/community/contribute Would you be okay with this? If so, please let me know once your CLA has been received and processed.
Hide
Permalink
Paul Mitchell added a comment -

Hi Trevor,

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.

Show
Paul Mitchell added a comment - Hi Trevor, 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.
Hide
Permalink
Roy Ganor added a comment -

my temportary context

Show
Roy Ganor added a comment - my temportary context

People

Vote (0)
Watch (1)

Dates

  • Created:
    Updated:
Atlassian JIRA (v4.4.3#663-r165197) | Report a problem
Powered by a free Atlassian JIRA open source license for Zend Framework. Try JIRA - bug tracking software for your team.