Skip to content
Snippets Groups Projects

Add timestamp columns to VSA aggregated RecordsFetcher

All threads resolved!

Resolves #364361 (closed)

The aggregated version of the VSA RecordsFetcher, used at group level, was missing the start_event_timestamp and end_event_timestamp columns. As a result, such attributes were always null in the JSON response to Projects::Analytics::CycleAnalytics::StagesController#records.

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Related to #364361 (closed)

Edited by Pavel Shutsin

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • requested review from @ahegyi

  • added 1 commit

    • 0b5988a9 - Add start_event_timestamp and end_event_timestamp in order_by_duration

    Compare with previous version

  • Adam Hegyi approved this merge request

    approved this merge request

  • :wave: @ahegyi, thanks for approving this merge request.

    This is the first time the merge request is approved. To ensure full test coverage, a new pipeline has been started.

    For more info, please refer to the following links:

  • requested review from @stanhu

  • Magdalena Frankiewicz removed review request for @stanhu

    removed review request for @stanhu

  • requested review from @pshutsin

  • Pavel Shutsin resolved all threads

    resolved all threads

  • Pavel Shutsin approved this merge request

    approved this merge request

  • Pavel Shutsin marked the checklist item I have evaluated the MR acceptance checklist for this MR. as completed

    marked the checklist item I have evaluated the MR acceptance checklist for this MR. as completed

  • Pavel Shutsin enabled an automatic merge when the pipeline for a5cca9c9 succeeds

    enabled an automatic merge when the pipeline for a5cca9c9 succeeds

  • Pavel Shutsin removed review request for @pshutsin

    removed review request for @pshutsin

  • merged

  • Pavel Shutsin mentioned in commit 7b4e9849

    mentioned in commit 7b4e9849

  • added workflowstaging label and removed workflowcanary label

  • Please register or sign in to reply
    Loading