summary-osv #2
maxim
Loading…
x
Reference in New Issue
Block a user
No description provided.
Delete Branch "summary-osv"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Не особо смотрел на валидность sql-функции, но вроде выглядит правильно
@ -0,0 +14,4 @@def upgrade() -> None:# PostgreSQL requires recreation when RETURNS TABLE changes.# No CASCADE: dependent database objects must not be silently removed.op.execute("DROP FUNCTION dkb.get_osv_entries_for_group(uuid, integer, text)")Надо избавиться от указания схемы в создании функции и доступа к таблицам (в alembic/env.py в миграции теперь сначала происходит
set search_path to "{db_schema}";, поэтому в целом должно прокатить просто удаление 'dkb.' из всей миграции, при этом в ENV-переменных будем прописывать схему. Та же схема используется в src/db/db/engine (search_path=db_schema)исправила везде хард код схемы на settings.OPENBAO.DATABASE.DATABASE_SCHEMA
А зачем здесь вообще схему оставлять? У нас в db.engine проставляется search_path из env, что эквивалентно, тут как будто не нужна схема вообще. Или без нее не работает?
@ -0,0 +20,4 @@def upgrade() -> None:op.execute("""CREATE OR REPLACE FUNCTION dkb.get_osv_entries_for_group(Тот же комментарий про схему
@ -12,0 +15,4 @@async def get_by_company_group(self, company_group_id: uuid.UUID, year: int, quarter: PeriodQuarter) -> list[OsvGroupEntryResponse]:entries = await self.osv_repo.get_by_company_group(.model_validate может упасть с ошибкой, возможно, стоит как-то обработать