Skip to content
Snippets Groups Projects

WIP: Use a new table for user contribution stats

Closed ido leibovich requested to merge leibo/gitlab-ce:user-contribution-table into master

What does this MR do?

It defers the user contribution stats to a new table, instead of the events table.

Are there points in the code the reviewer needs to double check?

Why was this MR needed?

This will allow down sizing the events table.

Screenshots (if relevant)

Does this MR meet the acceptance criteria?

What are the relevant issue numbers?

#22623 (moved)

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
  • @leibo I posted some comments on the migration above.

  • ido leibovich Added 1 commit:

    Added 1 commit:

    • 0cd5c1c1 - Use a new table for user conribution stats

    Compare with previous version

  • Author Contributor

    @yorickpeterse Hi, I fixed according to your comments, WDYT?

  • Reassigned to @leibo

  • ido leibovich Added 1 commit:

    Added 1 commit:

    • 76160d06 - Use a new table for user conribution stats

    Compare with previous version

  • ido leibovich Resolved all discussions

    Resolved all discussions

  • Author Contributor

    @yorickpeterse fixed, moving on I think

  • @leibo Looks good so far :thumbsup:

  • Mentioned in issue #24344 (closed)

  • Clement Ho Added ~164274 label

    Added ~164274 label

  • ido leibovich Added 1 commit:

    Added 1 commit:

    • 31c2b3b1 - Use a new table for user conribution stats

    Compare with previous version

  • Author Contributor

    @yorickpeterse Added the model and Sidekiq worker, take a look.

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