Skip to content

Resolve "Extend metric tracking from diffs_batch to diffs_path"

What does this MR do and why?

We are now loading file-by-file, which means we don't use the diffs_batch endpoint anymore, and this tracking was being skipped. We need to continue tracking this when we load the diffs file-by-file too.

To test:

  1. Check the current count of MR's Gitlab::UsageDataCounters::HLLRedisCounter.unique_events(event_names: 'i_code_review_mr_single_file_diffs', start_date: Date.current.beginning_of_week, end_date: Date.current.next_week)
  2. Change profile settings to - one file at a time
  3. Load an MR diffs tab
  4. Check the count has increased by one

Related to #413091 (closed)

Edited by Marc Shaw

Merge request reports

Loading