search

fi-ksi / web-backend

Backend for KSI web written in Python.
MIT License
8 stars 3 forks source link

fix(runcode): truncate overflowing reports #142

Closed esoadamo closed 2 years ago

esoadamo commented 2 years ago

Resolves https://github.com/fi-ksi/web-backend/issues/141

@BorysekOndrej Prosím, podívej se, jestli souhlasíš s řešením, nebo tě napadne něco elegantnějšího

BorysekOndrej commented 2 years ago

Jinak za mě ok.

BorysekOndrej commented 2 years ago

Teda vlastně - chceme vůbec do DB ukládat něco tak velkého?

640kb ought to be enough for anybody

Už teďka máme v DB občas problém, že některé selecty a views timeoutují, protože tahají příliš mnoho dat.

esoadamo commented 2 years ago

Jop, souhlasím, že to tak dlouhé nemá smysl. Takže na kolik? 640KiB?

BorysekOndrej commented 2 years ago

640KiB je dost náhodné číslo (resp. pochází s výše citovaného hlášky připisované (možná neprávem) Bill Gatesovi)). Na druhou stranu - proč ne. Zhruba 0,5 MiB mi přijde jako rozumný kompromis a 640 KiB je dostatečně blízko, abychom to mohli klidně nechat jako easter egg. :) Takže jsem pro.