Skip to content

WIP: Resolve "De-duplicate diff lines from backend"

Simon Knox requested to merge 50290-diff-lines-frontend into master

What does this MR do?

Performance improvement for MR Diffs. Remove parallel_diff_lines from diff_file entity and compute on frontend. Also saves a loop over the diff lines for trimFirstCharOfLineContent.

Convert parallelize function from parallel_diff.rb to utils.js

Sample MR with 100 changed files. Reduced http://localhost:3000/Commit451/lab-coat/merge_requests/3/diffs.json from 4.9MB -> 2MB

What are the relevant issue numbers?

Closes #50290 (moved)
Closes #48886 (closed)

Does this MR meet the acceptance criteria?

Edited by Chad Woolley

Merge request reports