Initial swap from redis to postgres #4612

barthalion · 2025-02-08T09:44:18Z

No description provided.

backend/app/summary.py

backend/app/app.py

razzeee · 2025-02-08T13:53:22Z

Some small nitpicks, seems fine otherwise

bbhtt · 2025-02-08T14:31:47Z

Code seems fine to me.

backend/app/app.py

backend/app/worker.py

barthalion added 7 commits February 8, 2025 10:37

models: Add appstream and summary columns to Apps

917a43d

worker: Read created_at from the database

e90a744

Change /summary to read data from postgres

561a064

apps: Load data to postgres

cf6456f

Use postgres for /addon

c21701d

moderation: Read summary from postgres

808d86d

summary: Store data in postgres

290aa64

razzeee reviewed Feb 8, 2025

View reviewed changes

backend/app/summary.py Outdated Show resolved Hide resolved

razzeee reviewed Feb 8, 2025

View reviewed changes

backend/app/summary.py Show resolved Hide resolved

razzeee reviewed Feb 8, 2025

View reviewed changes

backend/app/app.py Outdated Show resolved Hide resolved

bbhtt reviewed Feb 8, 2025

View reviewed changes

backend/app/app.py Outdated Show resolved Hide resolved

bbhtt reviewed Feb 8, 2025

View reviewed changes

backend/app/worker.py Show resolved Hide resolved

bbhtt reviewed Feb 8, 2025

View reviewed changes

backend/app/worker.py Show resolved Hide resolved

barthalion added 4 commits February 8, 2025 22:04

summary: Remove unnecessary row rewrite

388f460

app: Rename "value" to "summary" for clarity

d557b53

Simplify multi-condition if

196df7d

Be more defensive about created_at

41446a9

barthalion merged commit 4abdb07 into main Feb 9, 2025
2 checks passed

barthalion deleted the redis-to-postgres branch February 9, 2025 07:45

Provide feedback