search

manGoweb / MangoPress

(WordPress + Nette)²
35 stars 4 forks source link

Pr/sentry in config #68

Closed FilipChalupa closed 7 years ago

FilipChalupa commented 7 years ago

Fixes https://github.com/manGoweb/MangoPress/issues/67

ViliamKopecky commented 7 years ago

Ten neon se bude parsovat při každým requestu. Blbé je, že to sentry musí běžet asi ještě před vytvářením DI containeru.

Co mít v deploy.sh skript, co to z neonu vytáhne jednou a uloží do toho původního jsonu?

FilipChalupa commented 7 years ago

@ViliamKopecky Má smysl zůstat u jsonu, když se úplně stejně bude muset parsovat při každém requestu? S ním si tedy moc nepomůžeme.

Nevím, jak do bashe dostat neon decoder. Měl bys případně nějaký nápad?

ViliamKopecky commented 7 years ago

Tak jsem si ozkoušel rychlostní rozdíl. A imho je to zanedbatelné. Takže nechme takhle :)

100000×
neon: 23.93214 s
json: 6.685424 s

1×
neon: 0.0015 s
json: 0.0005 s

Jinak v bashi můžeš pustit php skript...

ViliamKopecky commented 7 years ago

Ještě by chtělo vyřešit situaci u zakládání serveru - teď to generuje ten json. Místo toho asi generovat rovnou tu neonovou část do local configu (tak jako to generuje db připojení).

ViliamKopecky commented 7 years ago

@Onset Já už jsem to rebasnul bez toho commitu s composerem, a ted jsi to tam zas přidal - já bych to tam vůbec nedával :)

FilipChalupa commented 7 years ago

Dík za test.

Do serverové části jsem se nechtěl zatím pouštět, pokud by tenhle PR nedával smysl. Jdu na to.

FilipChalupa commented 7 years ago

Ok, neon jsem zahodil úplně.

FilipChalupa commented 7 years ago

@ViliamKopecky Přidal jsem PR do sysops a default repo, které doplňují kompatibilitu s novou strukturou.