Skip to content
Snippets Groups Projects

Add write buffer to AiUserMetrics model

What does this MR do and why?

Adds write buffer for AI::UserMetrics model. It will be used to schedule delayed updates to user metrics.

No user facing changes yet. It's only a preparation MR to have smaller MRs.

MR acceptance checklist

Please evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.

Screenshots or screen recordings

Screenshots are required for UI changes, and strongly recommended for all other merge requests.

Before After

How to set up and validate locally

Numbered steps to set up and validate the change are strongly suggested.

Related to #512247 (closed)

Edited by Pavel Shutsin

Merge request reports

Merge train pipeline #1654975188 passed

Merge train pipeline passed for f57c4ccb

Approval is optional

Merged by Jan ProvaznikJan Provaznik 1 month ago (Feb 4, 2025 12:43pm UTC)

Merge details

  • Changes merged into master with 8b4bd4f5 (commits were squashed).
  • Deleted the source branch.
  • Auto-merge enabled

Pipeline #1654977108 passed

Pipeline passed for 8b4bd4f5 on master

10 environments impacted.

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • 1 Message
    :book: CHANGELOG missing:

    If this merge request needs a changelog entry, add the Changelog trailer to the commit message you want to add to the changelog.

    If this merge request doesn't need a CHANGELOG entry, feel free to ignore this message.

    Reviewer roulette

    Category Reviewer Maintainer
    backend @olaoluro profile link current availability (UTC+0, 1 hour behind author) @aakriti.gupta profile link current availability (UTC+1, same timezone as author)
    database @irina.bronipolsky profile link current availability (UTC+0, 1 hour behind author) @terrichu profile link current availability (UTC-5, 6 hours behind author)
    ~"Clickhouse" Reviewer review is optional for ~"Clickhouse" @pedropombeiro profile link current availability (UTC+1, same timezone as author)

    Please refer to documentation page for guidance on how you can benefit from the Reviewer Roulette, or use the GitLab Review Workload Dashboard to find other available reviewers.

    Sidekiq queue changes

    This merge request contains changes to Sidekiq queues. Please follow the documentation on changing a queue's urgency.

    These queues were added:

    • cronjob:analytics_dump_ai_user_metrics_write_buffer_cron

    If needed, you can retry the :repeat: danger-review job that generated this comment.

    Generated by :no_entry_sign: Danger

    Edited by ****
  • Pavel Shutsin changed title from Create AiUserMetrics table and model to Add write buffer to AiUserMetrics table

    changed title from Create AiUserMetrics table and model to Add write buffer to AiUserMetrics table

  • Pavel Shutsin changed title from Add write buffer to AiUserMetrics table to Add write buffer to AiUserMetrics model

    changed title from Add write buffer to AiUserMetrics table to Add write buffer to AiUserMetrics model

  • Pavel Shutsin added 1 commit

    added 1 commit

    • 452b5c8e - Add WriteBuffer for Ai::UserMetrics

    Compare with previous version

  • E2E Test Result Summary

    allure-report-publisher generated test report!

    e2e-test-on-cng: :white_check_mark: test report for 2d953a22

    expand test summary
    +------------------------------------------------------------------+
    |                          suites summary                          |
    +-------------+--------+--------+---------+-------+-------+--------+
    |             | passed | failed | skipped | flaky | total | result |
    +-------------+--------+--------+---------+-------+-------+--------+
    | Package     | 29     | 0      | 15      | 0     | 44    | ✅     |
    | Create      | 140    | 0      | 22      | 0     | 162   | ✅     |
    | Verify      | 53     | 0      | 19      | 0     | 72    | ✅     |
    | Plan        | 86     | 0      | 8       | 0     | 94    | ✅     |
    | Govern      | 84     | 0      | 10      | 0     | 94    | ✅     |
    | Secure      | 2      | 0      | 5       | 0     | 7     | ✅     |
    | Monitor     | 8      | 0      | 12      | 0     | 20    | ✅     |
    | Data Stores | 33     | 0      | 10      | 0     | 43    | ✅     |
    | Manage      | 1      | 0      | 9       | 0     | 10    | ✅     |
    | Growth      | 0      | 0      | 2       | 0     | 2     | ➖     |
    | Analytics   | 2      | 0      | 0       | 0     | 2     | ✅     |
    | Fulfillment | 2      | 0      | 7       | 0     | 9     | ✅     |
    | Ai-powered  | 0      | 0      | 2       | 0     | 2     | ➖     |
    | Configure   | 0      | 0      | 3       | 0     | 3     | ➖     |
    | Release     | 5      | 0      | 1       | 0     | 6     | ✅     |
    | ModelOps    | 0      | 0      | 1       | 0     | 1     | ➖     |
    +-------------+--------+--------+---------+-------+-------+--------+
    | Total       | 445    | 0      | 126     | 0     | 571   | ✅     |
    +-------------+--------+--------+---------+-------+-------+--------+

    e2e-test-on-gdk: :white_check_mark: test report for 2d953a22

    expand test summary
    +------------------------------------------------------------------+
    |                          suites summary                          |
    +-------------+--------+--------+---------+-------+-------+--------+
    |             | passed | failed | skipped | flaky | total | result |
    +-------------+--------+--------+---------+-------+-------+--------+
    | Govern      | 37     | 0      | 2       | 0     | 39    | ✅     |
    | Create      | 35     | 0      | 0       | 0     | 35    | ✅     |
    | Plan        | 14     | 0      | 0       | 0     | 14    | ✅     |
    | Verify      | 6      | 0      | 0       | 0     | 6     | ✅     |
    | Package     | 4      | 0      | 0       | 0     | 4     | ✅     |
    | Analytics   | 1      | 0      | 0       | 0     | 1     | ✅     |
    | Data Stores | 12     | 0      | 0       | 0     | 12    | ✅     |
    | Secure      | 1      | 0      | 0       | 0     | 1     | ✅     |
    +-------------+--------+--------+---------+-------+-------+--------+
    | Total       | 110    | 0      | 2       | 0     | 112   | ✅     |
    +-------------+--------+--------+---------+-------+-------+--------+
    Edited by ****
  • Pavel Shutsin added 1 commit

    added 1 commit

    • 37a2cc1d - Add WriteBuffer for Ai::UserMetrics

    Compare with previous version

  • Pavel Shutsin added 650 commits

    added 650 commits

    Compare with previous version

  • Pavel Shutsin added 1 commit

    added 1 commit

    • d518ff26 - Add WriteBuffer for Ai::UserMetrics

    Compare with previous version

  • Pavel Shutsin added 1 commit

    added 1 commit

    • 855e1311 - Add WriteBuffer for Ai::UserMetrics

    Compare with previous version

  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading