Skip to content

Comments

☕ Coffee-free Database#1

Merged
williamstein merged 67 commits intomainfrom
decaff-db
Feb 7, 2026
Merged

☕ Coffee-free Database#1
williamstein merged 67 commits intomainfrom
decaff-db

Conversation

@haraldschilly
Copy link
Contributor

@haraldschilly haraldschilly commented Jan 20, 2026

This PR is a cherry-picked and conflict-resolved transplant of sagemathinc/cocalc#8687, but only up to commit ea897443a1f01f00e90924542ee61acd291649c2.

…d adjust tests -- issue appeared after merging in the diverging changes from branch project-transfer-ownership-7718
- drop _clients/_client_index and make pg.Pool the core connection for PostgreSQL

- remove per-instance connection options and assume shared pool config

- add _get_query_client/_get_listen_client and keep a dedicated LISTEN/NOTIFY client

- route queries through pooled clients and update connect/disconnect semantics

- adjust sync/message/group helpers to release pooled clients

- remove backup/restore ops from database API while keeping getBackupTables/NON_CRITICAL_TABLES

- update tests and cleanup to close listen clients and await pool shutdown
@haraldschilly
Copy link
Contributor Author

haraldschilly commented Jan 23, 2026

Author: Codex (comment written by me)

Process

Findings and changes applied in decaff-db

  • Removed Stripe syncCustomer on change-email (CoffeeScript removed it in main).
  • Removed project-upgrades logic in user-set project sanitization and hooks; no more upgrade-based quota actions.
  • Removed upgrade-based checks from project network access and active student stats; keep member_host settings only.
  • Updated tests accordingly.
  • Exposed membership tiers on db interface (membershipTiers) to align with the membership-based fork direction.
  • pglite/UTC/normalize logic already present in TS, no further changes needed.

Commit: f997e8e (database/postgres: drop upgrades and stripe sync).

@williamstein williamstein merged commit 0694f71 into main Feb 7, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants