cyrussmeat
  • Joined on 2022-07-14
cyrussmeat commented on pull request Vorkout/connect#22 2025-11-24 19:54:37 +05:00
feat: settings, delete port validation

С т.з. репозитория можно удалить в этой версии, мы всегда можем вернуться к этой ветке и позаимствовать код,…

cyrussmeat approved Vorkout/core#4 2025-11-24 19:46:03 +05:00
WIP: feat: delete ports
cyrussmeat approved Vorkout/connect#21 2025-11-07 17:18:57 +05:00
feat: endpoints services
cyrussmeat commented on pull request Vorkout/connect#21 2025-11-07 17:02:47 +05:00
feat: endpoints services

Да, Под каждый ендпоинт, свой класс.

Окей, мне просто показалось, что сервисы могут иметь более широкий…

cyrussmeat commented on pull request Vorkout/connect#21 2025-11-07 12:31:58 +05:00
feat: endpoints services

Стало сильно лучше. Вопрос, по организации папок только services/endpoints типа ты руководствовался тем, что это…

cyrussmeat approved Vorkout/core#3 2025-10-30 08:14:51 +05:00
feat: CASCADE node link table
cyrussmeat approved Vorkout/connect#20 2025-10-30 08:14:28 +05:00
feat: delete ps node
cyrussmeat commented on pull request Vorkout/connect#20 2025-10-28 16:31:01 +05:00
feat: delete ps node

Это предмет крупного рефакторинга. В основном наблюдение говорит об избыточности семантики в названиях…

cyrussmeat commented on pull request Vorkout/connect#20 2025-10-23 17:15:48 +05:00
feat: delete ps node

С этим трудно спорить. В целом, если учитывать латентность сетевую и парсинг SQL, оно уложится в 0.1-0.3сек на…

cyrussmeat commented on pull request Vorkout/connect#20 2025-10-23 16:11:50 +05:00
feat: delete ps node

Это LLM, мне кажется )

cyrussmeat commented on pull request Vorkout/connect#20 2025-10-23 16:09:59 +05:00
feat: delete ps node

Ещё момент. Циклы у нас могут быть, когда замыкающая цикл нода ссылается куда-то "наверх".

Это решается…

cyrussmeat commented on pull request Vorkout/connect#20 2025-10-23 16:07:30 +05:00
feat: delete ps node

Особой нагрузки тут не предвидится, как минимум в разрезе редактирования тела схемы процесса. Речь про…

cyrussmeat suggested changes for Vorkout/connect#20 2025-10-23 10:32:21 +05:00
feat: delete ps node
cyrussmeat commented on pull request Vorkout/connect#20 2025-10-23 10:32:21 +05:00
feat: delete ps node

Там FK у node_link на node_id (т.е. нужно удалять node_link раньше node_id, но позже next_node_id), есть у меня сомнение, что такое упорядочивание здесь реализовано.

cyrussmeat approved Vorkout/connect#19 2025-10-15 19:20:05 +05:00
feat: CRUD process schema
cyrussmeat suggested changes for Vorkout/connect#19 2025-10-15 17:17:19 +05:00
feat: CRUD process schema
cyrussmeat commented on pull request Vorkout/connect#19 2025-10-15 17:17:19 +05:00
feat: CRUD process schema

Это ты эмулируешь last_insert_id = await database.execute(query) ?

cyrussmeat commented on pull request Vorkout/connect#19 2025-10-15 17:17:19 +05:00
feat: CRUD process schema

Я поглядел, что реализация в process_schema не задействует данный метод. Он нужен?

cyrussmeat commented on pull request Vorkout/connect#19 2025-10-15 17:17:19 +05:00
feat: CRUD process schema

А почему в такой связке не используется last_insert_id и получение схемы по идентификатору из БД, который PK?

cyrussmeat approved Vorkout/core#2 2025-10-13 10:47:02 +05:00
VORKOUT-18