kunwp1 opened a new pull request, #6306:
URL: https://github.com/apache/texera/pull/6306

   ### What changes were proposed in this PR?
   
   Bulk-uploading many files to a dataset was extremely slow (~1,100 files took 
~12 minutes) because the Pending and Finished lists fully re-rendered on every 
change-detection pass, on the same thread driving the uploads (root cause 
detailed in #5586).
   
   As proposed in the issue: replace the `queuedFileNames` getter with a cached 
field refreshed only when the queue changes, virtualize the Pending and 
Finished lists with `cdk-virtual-scroll` so only visible rows are in the DOM, 
and add `trackBy` to both lists.
   
   Screen recording of a successful 1,100-file upload with the fix:
   
   ### Any related issues, documentation, discussions?
   
   Closes #5586.
   
   ### How was this PR tested?
   
   New unit specs (12 tests): `npx ng test --watch=false --include 
"**/dataset-detail.component.spec.ts" --include 
"**/user-dataset-staged-objects-list.component.spec.ts"`. Full dashboard suite: 
`npx ng test --watch=false --include "src/app/dashboard/**/*.spec.ts"` — 346 
passed. Manually verified on the local dev stack: 1,100 files upload 
successfully with the UI responsive throughout (recording above).
   
   ### Was this PR authored or co-authored using generative AI tooling?
   
   Generated-by: Claude Code, Claude Fable 5
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to