Get rid of user activites table and replace it with redis
- Gets rid of the user activities table
- Switch from postgreSQL to Redis
Merge request reports
Activity
@yorickpeterse I'll really appreciate if you can do an initial review of this MR... This one is mainly to get rid of the user activity stuff.
I'm not sure about the redis bit. Is
Gitlab::Redis
using the default cache namespace? Shall I use something else for this?I'll open up a new MR for the API and reading the data bit as this is just so we can keep recording activities.
Reassigned to @yorickpeterse
- Resolved by Yorick Peterse
- Resolved by James Lopez
- Resolved by James Lopez
- Resolved by James Lopez
Reassigned to @jameslopez
- Resolved by James Lopez
Mentioned in issue #1288 (closed)
Added 1 commit:
- 3da15069 - Fixed spec, removed duplicate push count, also updated code based on feedback
Added 1 commit:
- a6226b6a - add extra spec for testing the actual time change in an update
@yorickpeterse updated the code. Can you review again? Also added a spec to double-check we are updating users and not creating new entries.
Edited by James LopezReassigned to @yorickpeterse
- Resolved by James Lopez
@yorickpeterse quick question. Do you know when does the
Gitlab::Redis
cache expire (if ever?) and is there any task or background job that runs that could clean/remove theuser/activities
set ?- Resolved by Yorick Peterse