Ticket #360 (closed enhancement: fixed)

Opened 3 years ago

Last modified 22 months ago

/assets REST optimize.

Reported by: robert Owned by: robert
Priority: major Milestone: MediaMosa 3.0
Component: REST Version:
Keywords: Cc:
MoSCoW: Must Have Estimated time after impact analysis:
Related to project: none Tested: no
Accepted: no Estimated Hours: 0

Description

The /assets/ REST call can be slow when providing it with a lot of assets to collect. The reason is the ACL check, which is done on every asset separately. Fix is by providing the ACL check with all asset ids and use the returned asset_ids instead (which passed ACL).

Change History

Changed 2 years ago by Frans

  • milestone changed from MediaMosa X.X to MediaMosa 2.4

Relevant for OAI and CSA

Changed 2 years ago by Frans

  • moscow changed from Should Have to Must Have

Changed 2 years ago by forgacs

  • owner set to forgacs
  • status changed from new to assigned

Changed 2 years ago by forgacs

  • status changed from assigned to closed
  • resolution set to fixed

/assets REST call optimize is done.
Optimize 1: ACL check was moved to CQL search.
Optimize 2: mediamosa_asset::asset_collect() function is using asset_ids array now, and it was moved out from the foreach().

Changed 2 years ago by robert

  • status changed from closed to reopened
  • resolution fixed deleted

Changed 2 years ago by robert

  • owner changed from forgacs to robert
  • status changed from reopened to assigned

Changed 2 years ago by robert

  • status changed from assigned to closed
  • resolution set to fixed

Changed 22 months ago by forgacs

There isn't really good test for this.
You can measure the speed with REST calls like this:
/assets?asset_id[]=vM8WljsiZFjnLQlMfJLvVQmS&asset_id[]=s3mfUbXDNrNiPYsTeKwfdl73

Note: See TracTickets for help on using tickets.