Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Mysql/Aurora support for Frappe #11812

Closed
wants to merge 1 commit into from
Closed

Mysql/Aurora support for Frappe #11812

wants to merge 1 commit into from

Conversation

seethersan
Copy link
Contributor

Please provide enough information so that others can review your pull request:
Gives Frappe compatibility with Aurora/Mysql database by setting the db_type option in the site's site_config.json file as follows:

 ...
 "db_type": "mysql"
...

You can also add it in the new-site command

bench new-site --db-type mysql site1.local

Explain the details for making this change. What existing problem does the pull request solve?
This change allows the use of ERPNext with AWS Aurora/Mysql.

Screenshots/GIFs

@ghost ghost removed their request for review October 28, 2020 17:35
@rmehta
Copy link
Member

rmehta commented Nov 2, 2020

how is this different from MariaDB?

Would it be better to subclass mariadb/database.py for fixes?

@seethersan
Copy link
Contributor Author

The difference is the use of COMPRESS format in table creation because it isn't compatible with Aurora/Mysql. The main reason I used this approach is to allow the use of this solution as an option in db_type inside site_config file.

@bitgandtter
Copy link

It feels less disruptive at been an opt-in feature

@rmehta
Copy link
Member

rmehta commented Nov 17, 2020

@seethersan closing this since this has significant code repetition. The subclass must only update the methods that change. Please reopen after fixing.

@rmehta rmehta closed this Nov 17, 2020
@bitgandtter
Copy link

Is there any intention of allow usage of other database like MySQL 8 or aurora?

@rmehta
Copy link
Member

rmehta commented Nov 17, 2020 via email

@bitgandtter
Copy link

We are in need to be able to switch to mysql8 or aurora, we need managed databases and sadly not all managed database providers have mariadb on their stock. If you or anyone let us know what we can do to help move on that direction we will happy do it

@rmehta
Copy link
Member

rmehta commented Nov 18, 2020

@bitgandtter make sure you follow contribution guidelines correctly or find someone who can do this the right way. I can recommend @revant ?

@revant
Copy link
Collaborator

revant commented Nov 19, 2020

Search the frappe/erpnext code base for "mariadb" or 'mariadb', there are many places where it is used.
Also in many places frappe.db.multisql() is used. What's the plan regarding that?

@bitgandtter to use Aurora refer this https://discuss.erpnext.com/t/erpnext-installation-in-rds-aurora-mysql/48596/6

@nikoma
Copy link

nikoma commented May 19, 2022

I see that this is 2 years old. Has this been integrated at some point? We are planning a large rollout and would love to use Amazon Aurora.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 19, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants