Cumulus/Cumulus — pull request #40 by jpdeplaix (2013-11-23 22:57:58+00:00)
Title: Optimization: DO NOT PUSH

Note: Je fais cette PR pour qu'on puissent commenter (et poser quelques questions) au fur et à mesure.

D'abord, je trouve que Db_table est une très bonne idée, mon commentaire se trouvera plus pour ce qui est dans Db_all.

* Donc, déjà je ne comprend pas pourquoi tu veux utiliser absolument les fonctions de Lwt_list alors que le module List suffis. Ça alourdis la syntaxe, c'est dommage :/
* Je ne vois toujours pas à quoi sert le `Sql.Op.(+) value#score <:value<1>>`. Pourrais-tu apporter tes lumières là-dessus ?
* Question de goût, mais j'aurais plus vu un Set au lieux du couple `find` + `filter`.

Enfin si, je crois voir pour le deuxième point, mais t'es sûr que c'est pas plutôt `Sql.Op.(+) value#score element#score` ?