Backup: заменить JSON-клонирование снимка восстановления на типизированную копию

Aperta
  • Qualcosa non va?
#393 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 1 giorno

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
5/5
Tempo stimato
Più di una settimana
Idoneità per principianti
48/100
Tipo di issue
Refactoring
Chiarezza
Specificata chiaramente
Stato di attività
Attiva
Stack tecnologico
csharp
Ambito
backend

Direzione di ricerca

Start at ProfileStore.RestoreBackupAsync and BackupArchive.Clone(data), then inspect the backup models and the existing Merge/Replace, save failure, cancellation, and rollback tests. Implement an explicit copy that preserves every model field while giving mutable collections and nested models independent containers. Done means the listed isolation and restore tests pass, the full backup clone path avoids JSON serialization, and a repeatable benchmark shows lower allocations on both datasets.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

Проблема

ProfileStore.RestoreBackupAsync вызывает BackupArchive.Clone(data). Клонирование сериализует весь BackupData в JSON и сразу десериализует обратно. Это создаёт дополнительный plaintext-буфер с секретами и повторные строки, включая исходные конфигурации. В Clone временный JSON-массив явно не обнуляется.

Измерения

Изолированный синтетический бенчмарк, Windows x64, .NET 10.0.7 Release, tiered compilation выключена, два прогрева и медиана семи повторов:

  • 1000 профилей: JSON-клон — 5,20 мс и 4,03 МиБ аллокаций; типизированный C#-прототип — 0,34 мс и 0,76 МиБ.
  • 200 профилей с отдельными строками конфигураций по 128 КиБ; JSON архива около 25,1 МиБ: JSON-клон — 21,92 мс и 138,95 МиБ; прототип — 0,15 мс и 0,17 МиБ.

Это суммарные управляемые аллокации за операцию, не пик RAM и не время полного восстановления. В прототипе создаются независимые изменяемые контейнеры, но переиспользуются неизменяемые строки. Проверены равенство сериализованного результата и отсутствие общей ссылки на словарь секретов; это не заменяет полноценные тесты всех моделей.

Решение

Заменить JSON round-trip для снимка восстановления на явное копирование в C#. Копировать все изменяемые коллекции и вложенные модели; строки разрешено переиспользовать. Проверку данных, Merge/Replace, откат секретов и порядок сохранения оставить семантически прежними. Остальные вызовы общего Clone<T> менять только при необходимости для этого пути.

Готово, когда

  • Изменение исходных или скопированных словарей, списков, профилей, подписок, цепочек и настроек не меняет другой снимок.
  • Сохраняются все поля моделей, включая идентификаторы, ссылки на подписки, параметры маршрутизации и секреты инструментов.
  • На пути клонирования полного backup отсутствует промежуточная JSON-сериализация секретов.
  • Изолированные тесты покрывают Merge/Replace, ошибки сохранения, отмену и rollback; существующие гарантии восстановления сохраняются.
  • Повторяемый бенчмарк подтверждает снижение аллокаций на обоих описанных наборах без требования конкретного времени на другом оборудовании.

Связано с #18 и #181. Смена языка, формата .bydlobackup и криптографических параметров не требуется. Все проверки — с BYDLO_SAFE_DEV=1, фейковым хранилищем, без запуска движков и системного сетевого воздействия.

Lingua principale
C#
Stelle
5
Fork
1
Merge medio
30m
PR unite (30g)
171

Preparare l'ambiente

Questo progetto non fornisce container di sviluppo, Dockerfile né guida per i contributori, quindi l'ambiente è a tuo carico: parti dal suo README e consulta la nostra guida al primo contributo per i passaggi generali.

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di endercodezz/BYDLO

Tutte le issue di endercodezz/BYDLO

Issue simili

Altre issue su C#

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.