[EXTDAM-170] [173, 174] (S3) Improve rendering when there are thousand of assets in the same folder Created: 14/Oct/20  Updated: 14/Jun/22  Resolved: 11/Nov/20

Status: Closed
Project: External DAMs
Component/s: s3
Affects Version/s: 1.0, 1.0.1
Fix Version/s: 1.0.2

Type: Improvement Priority: Neutral
Reporter: Jesus Alonso Assignee: Jesus Alonso
Resolution: Fixed Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: 2.05h
Original Estimate: Not Specified

Issue Links:
Relates
relates to EXTDAM-364 [S3] Performance issue when using Ric... Open
relates to EXTDAM-173 [170, 174] (S3) Use offset/limit when... Closed
relates to EXTDAM-174 [170, 173] Implement view which is ab... Closed
relation
is related to MGNLUI-4348 Introduce drill down view beside tree... Closed
Template:
Acceptance criteria:
Empty
Task DoD:
[ ]* Doc/release notes changes? Comment present?
[ ]* Downstream builds green?
[ ]* Solution information and context easily available?
[ ]* Tests
[ ]* FixVersion filled and not yet released
[ ]  Architecture Decision Record (ADR)
Release notes required:
Yes
Documentation update required:
Yes
Epic Link: DAM Phase 2
Sprint: Add-Ons 49, Add-Ons 50
Story Points: 13

 Description   

Problem
When an an S3 folder containing thousand of assets is expanded, it takes many seconds to render the new assets.

Considerations

  • HierarchicalS3DataProvider.hasChildren calls getChildCount, which will have to count all the items in the folder. We probably should have a dedicate hasChildren method in S3AssetProvider.
  • HierarchicalS3DataProvider.fetchChildrenFromBackEnd does not honor the supplied query

Acceptance criteria

The folder is properly displayed, probably rendering assets in batches.



 Comments   
Comment by Jesus Alonso [ 16/Oct/20 ]

Notice vaadin has a bug that prevents lazy loading the assets in the TreeView. Thus, we can't just fix this issue with HierarchicalS3DataProvider.fetchChildrenFromBackEnd as we always receive offset=0, limit=numberOfAssets, meaning we have to return all the assets in one shot.

An alternative implementation to TreeView is being considered. 

Generated at Mon Feb 12 01:52:34 CET 2024 using Jira 9.4.2#940002-sha1:46d1a51de284217efdcb32434eab47a99af2938b.