Use PostgreSQL instead of MySQL (fixes #107)

[JimboJoe]

Problem

• MySQL is much less robust than PostgreSQL

Solution

• Migrate Nextcloud to PostgreSQL

PR Status

• Code finished.
• Tested with Package_check.
• Fix or enhancement tested.
• Upgrade from last version tested.
• Can be reviewed and tested.

Validation

---

Minor decision

• Upgrade previous version :
• Code review :
• Approval (LGTM) :
• Approval (LGTM) :
• CI succeeded :
  

When the PR is marked as ready to merge, you have to wait for 3 days before really merging it.

[JimboJoe]

As I mentioned "WIP", I pushed that now just to illustrate the current API differences with MySQL helpers. .. Thanks for spotting the glitches anyway 😉

[maniackcrudelis]

Hopefully, the Urban Dictionnary is always here to save me !
https://www.urbandictionary.com/define.php?term=wip

[rushsteve1]

Any progress on this? Switching to Postgres would be a big improvement.

[JOduMonT]

FYI:

Yes I'm agreed PostgreSQL is more robust and perform better than mySQL but
and I don't remember by heart some nextcloud apps just won't works with PosgreSQL

also most of yunohost user focus more on usability and won't really care which database in background.

This why I think being able to use PostgreSQL should be an option;

[autra]

@JOduMonT do you remember which apps wouldn't work with postgres?

[JOduMonT]

@aura nope :(
I gave I try to google and nextcloud forum they did'nt remember too :P

[autra]

Ok, because I have used owncloud with postgresql (before using yunohost) during at least 2 years without any issue...

[Bugsbane]

Seems that the mysql 4 byte support issue now causes problems with Nextcloud News, and possibly Mail as well (still investigating that one). :/

[kay0u]

Migration should works now 🎁

So what should we do now? Do we really want to migrate to PostgreSQL? @YunoHost-Apps/apps-group

[ericgaspar]

In restore script, don't we need to install PostgreSQL with app_dependencies $pkg_dependencies before restoring the PostgreSQL database?

[kay0u]

Yep, a fix is coming

[kay0u]

After a fresh install (from testing) I have this error during the migration:

Warning: In AbstractPostgreSQLDriver.php line 54:
Warning: An exception occurred while executing 'SELECT setval('oc_text_documents_id_seq', (SELECT MAX() FROM ))':
Warning: SQLSTATE[42601]: Syntax error: 7 ERROR: syntax error at or near ")"
Warning: LINE 1: ...ECT setval('oc_text_documents_id_seq', (SELECT MAX() FROM ))
Warning: ^
Warning: In PDOConnection.php line 83:
Warning: SQLSTATE[42601]: Syntax error: 7 ERROR: syntax error at or near ")"
Warning: LINE 1: ...ECT setval('oc_text_documents_id_seq', (SELECT MAX() FROM ))
Warning: ^
Warning: In PDOConnection.php line 78:
Warning: SQLSTATE[42601]: Syntax error: 7 ERROR: syntax error at or near ")"
Warning: LINE 1: ...ECT setval('oc_text_documents_id_seq', (SELECT MAX() FROM ))

Edit: I don't know why my package_check pass the upgrade test and not my ynh-dev

Edit: Migrate from from the commit c864e08 works, not from testing ><

[JimboJoe]

Thanks a lot!!:heart_eyes::heart_eyes:
Curiously, MySQL is still the recommended database...?!

[Limezy]

https://paste.yunohost.org/raw/iqazirusuk
Any idea of what's happening ? @ericgaspar

[ericgaspar]

!testme

[yunohost-bot]

🚀

[Limezy]

FYI, Running that branch, upgrading fails because it starts by trying to make a backup and dumping the "nextcloud" mySQL database, that obviously doesn't exist !

[ericgaspar]

!testme

[yunohost-bot]

🌻

[ericgaspar]

superseded by #581