Skip to content

Instantly share code, notes, and snippets.

@AlexVanderbist
Last active April 20, 2017 08:57
Show Gist options
  • Star 0 You must be signed in to star a gist
  • Fork 0 You must be signed in to fork a gist
  • Save AlexVanderbist/b491a11db9dd9334482b9476ef75745a to your computer and use it in GitHub Desktop.
Save AlexVanderbist/b491a11db9dd9334482b9476ef75745a to your computer and use it in GitHub Desktop.
Blog dag #58

Vandaag is het tijd om de puntjes op de i te zetten voor de twee nieuwe packages. Beiden hebben nog wat basic tests en documentatie nodig en moeten dan getagd worden. Gelukkig krijg ik hier wel wat hulp. Zo kunnen beide packages tegen de middag als v1.0.0 getagged worden. Tegen 13u zitten ze allebei al op 1.0.1, je mist bijna altijd wel iets.

Het is altijd leuk als je werk geapprecieerd wordt. Zo is de Flysystem integratie opgenomen in de officiële Laravel documentatie!

De rest van de dag probeer ik bij wijze van ultieme integratietest de Dropbox driver te gebruiken in het backup pakket. Iets dat ik waarschijnlijk zelf ook zal gebruiken zo. Dat ging op eerste zicht zonder problemen maar bij het oplijsten van de backups ging het mis. Zo duurde het redelijk lang (3 minuten) om gewoon een lijstje van 2 backups weer te geven... Na een beetje debuggen werd het duidelijk dat heel wat simpele filesystems calls (zoals bv. kijken of een bestand bestaat) meerdere keren werden uitgevoerd binnen de dezelfde backup collectie. Meerdere keren was in dit geval 90 keer voor 2 backups. Na wat zoeken leek mij de beste oplossing om de meest 'bloated' filesystem calls te cachen. O.a. bijvoorbeeld de getSize call die recursief ging opvragen of een bestand bestaat en daarna hoe groot het was. Dit liep al snel op tot een tiental API calls.

Na wat selectieve caching is dit het resultaat:

(links origineel, rechts na optimalisaties)

published:true
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment