Supporter PostgreSQL

[ghost]

Certains ont PostgreSQL et non MySQL d'installé sur leur serveur/hébergement. Postgre en plus de MySQL et sqlite, et le monde entier sera ravi 👍

[marienfressinaud]

Effectivement, il n'y a pas de raison de ne pas gérer PostgreSQL :)

[aledeg]

Est ce que quelqu'un a commencé à travailler dessus?

[marienfressinaud]

Vu comment c'est galère de gérer SQLite en plus de MySQL, je n'ai pas eu le courage de regarder. Mais on peut espérer qu'il soit plus facile de gérer Postgre

[aledeg]

Quels sont les problèmes avec SQLite?

[marienfressinaud]

Pleins de requêtes qui ne sont pas compatibles (si ce n'est toutes).

[aledeg]

ok, je pense dans ce cas que postgres sera plus facile. Dans mes souvenirs, la syntaxe est très proche de celle de MySql.
Si on commence à supporter plusieurs base de données, il serait peut être judicieux d'utiliser un ORM comme Doctrine pour nous éviter tous les problèmes de compatibilité des requêtes.

[aledeg]

Propel supporte nativement MySQL, PostgreSQL, SQLite, MSSQL et Oracle. =>http://propelorm.org/
Doctrine supporte nativement MySQL, PostgreSQL, SQLite, MSSQL, Sybase et Oracle => http://doctrine-dbal.readthedocs.org/en/latest/reference/configuration.html#connection-details

[Alkarex]

Qui serait intéressé d'utiliser PostgreSQL ?
Je pense que suite au travail pour SQLite, le support basique de PostgreSQL devrait être assez facile.
Il faudrait par contre un peu plus de travail pour ensuite optimiser les performances, ajouter l'indexation du texte, etc.
Mais est-ce qu'il y a de la demande et des volontaires pour utiliser / tester / optimiser ?

[marienfressinaud]

@Alkarex > j'ai eu des questions d'une personne de Framasoft à propos de FRSS et qui a donc des problématiques de performances. Je pense que dans tous les cas ce serait intéressant de gérer Postgre.

[Alkarex]

Si certains ont des problèmes de performances avec MySQL (ou SQLite), il serait intéressant de les remonter (avec entre autres une sortie EXPLAIN de ou des requêtes problématiques).
Je ne pense pas que PostgreSQL sera magiquement plus rapide que MySQL, mais c'est bien d'avoir une possibilité de plus.

[aledeg]

J'ai commencé à regarder comment intégrer postgres.
Je n'ai pas trop de temps en ce moment, mais je veux bien m'en occuper.

[aledeg]

J'ai commencé l'ajout de Postgres.
J'arrive à créer la base donnée. Par contre j'ai un problème avec le code actuel car les tables et les colonnes sont entourés de `. Ça ne fonctionne pas avec postgres.
Est ce que je les supprime? Est ce que je cré des fichiers spécifiques pour postgres?

[Alkarex]

Il faudrait regarder s'il existe une syntaxe alternative supportée par les
3 systèmes

[aledeg]

on peut écrire les requêtes sans les `. On les utilise juste au cas où on aies des mots réservé de SQL dans les noms. Mais comme on gère nous même les tables et les colonnes, je pense qu'on peut les enlever.

[Alkarex]

Apparemment, avec SET sql_mode='ANSI_QUOTES', on peut utiliser la syntaxe standard dans MySQL, qui est donc aussi compatible avec PostgreSQL (vérifier pour SQLite). http://dev.mysql.com/doc/refman/5.0/en/sql-mode.html#sqlmode_ansi_quotes

[Alkarex]

Sinon pour enlever les accents graves, il faudrait vérifier quels sont les préfixes et les noms d'utilisateurs que nous autorisons pour voir s'il est toujours justifié de les protéger.
Je n'arrive plus immédiatement à retrouver le commit où Marien (je crois) avait commencé cette protection.

[aledeg]

Sinon, je peux refaire les requêtes juste pour postgres

[Alkarex]

Qu'en dit @marienfressinaud ? Je crois que tu avais ajouté ces accents graves pour des problèmes de noms d'utilisateurs ou préfixes de base ?

[marienfressinaud]

Ah pardon, j'ai oublié de répondre ! À vrai dire je ne me souviens même plus de cet ajout… Au niveau des préfixes, tant que ça reste alphanumerique ça ne me semble pas plus dangereux que ça, si ?

L'idéal serait une syntaxe qui fonctionnerait pour les trois SGBD. Si ce n'est pas possible, je trouve que réécrire toutes les requêtes pour Postgre est vraiment très dommage (on passe par PDO qui est censé nous abstraire quand même une grosse partie du travail). Du coup enlever les accents graves me semble être la deuxième meilleure piste, mais il va falloir tester les préfixes en profondeur !

[woshilapin]

Pour l'instant, je gère mes RSS avec OwnCloud (qui fonctionne sur PostGreSQL) en attendant que FreshRSS passe sur PostGreSQL. Je veux bien être cobaye :-) N'hésitez pas à me contacter pour tester vos propositions !

[aledeg]

Peut-être faudrait il attendre la fin de #732 ?

[mro]

thx!

[anayrat]

👍 merci pour le travail!

[Alkarex]

@Damstre Je viens de commencer un patch pour PostgreSQL basé sur votre travail, partant de la branche /dev actuelle, et en essayant de limiter le nombre de changements https://github.com/Alkarex/FreshRSS/tree/PostgreSQL Je n'ai pas encore testé.
Commentaires, contributions, et tests bienvenus dans #1195

[Alkarex]

PostgreSQL has landed in the dev branch of FreshRSS :-) Tests welcome!

[marienfressinaud]

\o/

[Alkarex]

En plus de PostgreSQL, il faut aussi vérifier qu'il n'y a pas de nouveau bug avec MySQL et SQLite, donc les tests sont aussi bienvenus pour ces bases de données

[Alkarex]

For the PostgreSQL testers of the /dev version, I would particularly be interested in feedback about:

• Performances
• Tests with full-text search (SQL : Considérer l'indexation du texte complet #302)
  • when adding a GIN index https://www.postgresql.org/docs/current/static/textsearch-indexes.html
  • https://www.depesz.com/2011/02/19/waiting-for-9-1-faster-likeilike/
• The minimum version of PostgreSQL