[PR #1380] [MERGED] refactor(backend): db session locking to support exclusive mode #1469

New issue

Closed

opened 2026-02-25 21:38:22 +03:00 by kerem · 0 comments

kerem commented

2026-02-25 21:38:22 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/cypht-org/cypht/pull/1380
Author: @Shadow243
Created: 11/23/2024
Status: ✅ Merged
Merged: 1/28/2025
Merged by: @kroky

Base: master ← Head: revamp-db-session-storage

📝 Commits (6)

3001b96 Fix: Revamp db session storage to work in exclusive locking mode like native file session storage works
311a69b refactor(setup database script): Review and improve database schema setup with column migration logic
8bee386 refactor(db_session): retry mechanism and lock timeout to optimize session locking and concurrency
263bc3f refactor(sqlite): db session implementation
5bbf052 feat(database): implement database migrations
6911557 refactor(migration): refactor migration process

📊 Changes

14 files changed (+398 additions, -81 deletions)

View changed files

📝 config/database.php (+2 -2)
➕ database/migrations/mysql/20241209010200_add_hm_version_columns.sql (+1 -0)
➕ database/migrations/pgsql/20241209010200_add_hm_version_columns.sql (+2 -0)
➕ database/migrations/sqlite/20241209010200_add_hm_version_columns.sql (+2 -0)
➕ database/migrations/sqlite/20241209010201_add_lock_columns.sql (+2 -0)
➕ database/mysql_schema.sql (+19 -0)
➕ database/pgsql_schema.sql (+34 -0)
➕ database/sqlite_schema.sql (+20 -0)
📝 lib/session_db.php (+137 -5)
📝 scripts/setup_database.php (+147 -71)
📝 tests/phpunit/data/schema.sql (+7 -1)
➕ tests/phpunit/data/schema_postgres.sql (+12 -0)
➕ tests/phpunit/data/schema_sqlite.sql (+11 -0)
📝 tests/phpunit/run.sh (+2 -2)

📄 Description

Related task item89272

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/cypht-org/cypht/pull/1380 **Author:** [@Shadow243](https://github.com/Shadow243) **Created:** 11/23/2024 **Status:** ✅ Merged **Merged:** 1/28/2025 **Merged by:** [@kroky](https://github.com/kroky) **Base:** `master` ← **Head:** `revamp-db-session-storage` --- ### 📝 Commits (6) - [`3001b96`](https://github.com/cypht-org/cypht/commit/3001b96c605c6416a36f0e5426e81c18fc6734f0) Fix: Revamp db session storage to work in exclusive locking mode like native file session storage works - [`311a69b`](https://github.com/cypht-org/cypht/commit/311a69bd8d68913fee5ba8fa92e836ec14d07470) refactor(setup database script): Review and improve database schema setup with column migration logic - [`8bee386`](https://github.com/cypht-org/cypht/commit/8bee38644f40c706f80699050092b94d97246f8c) refactor(db_session): retry mechanism and lock timeout to optimize session locking and concurrency - [`263bc3f`](https://github.com/cypht-org/cypht/commit/263bc3f86e56f08cd348f284debf79d4a8160511) refactor(sqlite): db session implementation - [`5bbf052`](https://github.com/cypht-org/cypht/commit/5bbf05211af7faa2b9acc5292de3897bf6860f37) feat(database): implement database migrations - [`6911557`](https://github.com/cypht-org/cypht/commit/6911557f564de7a1c0e81bc654e1831539a7c8e2) refactor(migration): refactor migration process ### 📊 Changes **14 files changed** (+398 additions, -81 deletions) <details> <summary>View changed files</summary> 📝 `config/database.php` (+2 -2) ➕ `database/migrations/mysql/20241209010200_add_hm_version_columns.sql` (+1 -0) ➕ `database/migrations/pgsql/20241209010200_add_hm_version_columns.sql` (+2 -0) ➕ `database/migrations/sqlite/20241209010200_add_hm_version_columns.sql` (+2 -0) ➕ `database/migrations/sqlite/20241209010201_add_lock_columns.sql` (+2 -0) ➕ `database/mysql_schema.sql` (+19 -0) ➕ `database/pgsql_schema.sql` (+34 -0) ➕ `database/sqlite_schema.sql` (+20 -0) 📝 `lib/session_db.php` (+137 -5) 📝 `scripts/setup_database.php` (+147 -71) 📝 `tests/phpunit/data/schema.sql` (+7 -1) ➕ `tests/phpunit/data/schema_postgres.sql` (+12 -0) ➕ `tests/phpunit/data/schema_sqlite.sql` (+11 -0) 📝 `tests/phpunit/run.sh` (+2 -2) </details> ### 📄 Description Related task [item89272](https://avan.tech/item89272) --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>