Bug 746899 - On push operations, copyTrans runs too slowly
Summary: On push operations, copyTrans runs too slowly
Alias: None
Product: Zanata
Classification: Retired
Component: Performance
Version: 1.4.1
Hardware: All
OS: All
Target Milestone: ---
: ---
Assignee: Carlos Munoz
QA Contact: David Mason
Depends On:
Blocks: zanata-1.5.0
TreeView+ depends on / blocked
Reported: 2011-10-18 07:54 UTC by David Mason
Modified: 2012-03-07 00:07 UTC (History)
3 users (show)

Fixed In Version: 1.5.0
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Last Closed: 2012-03-07 00:07:00 UTC

Attachments (Terms of Use)
Proposed inversion of copytrans loops to iterate over TransUnits before locales (9.04 KB, patch)
2012-03-01 03:12 UTC, David Mason
no flags Details | Diff

Description David Mason 2011-10-18 07:54:13 UTC
Description of problem:
copyTrans runs too slowly

Version-Release number of selected component (if applicable):

How reproducible:

Steps to Reproduce:
1.Push from any client with copytrans enabled
Actual results:
It is slow

Expected results:
Slightly less slow

Additional info:

Comment 1 Carlos Munoz 2012-02-20 23:23:03 UTC
Refactored the copy trans service to perform less database accesses and avoid unnecessary invocations. The copy trans service is also being invoked directly after pushing a resource instead of using a Seam event.

Should be anywhere between 60-80% faster than before based on tests.


Comment 2 David Mason 2012-02-23 06:31:29 UTC
Verified in 1.5

Push operations with copytrans now take up to ~30% less time with my test data.

Comment 3 David Mason 2012-03-01 03:12:24 UTC
Created attachment 566721 [details]
Proposed inversion of copytrans loops to iterate over TransUnits before locales

Master branch has diverged from this, it is included here for reference (would likely be a messy merge if applied).

Note You need to log in before you can comment on or make changes to this bug.