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

Update to support MARIADB_READREPLICA_HOSTS #109

Closed
wants to merge 1 commit into from

Conversation

steveworley
Copy link
Contributor

Changes

  • Updates read replica variable usage

@nicksantamaria
Copy link
Contributor

@steveworley is this a breaking change? i.e. does it need to go out with the 5.x branch?

@GROwen GROwen added the question Further information is requested label Oct 10, 2022
@GROwen
Copy link
Contributor

GROwen commented Oct 10, 2022

@steveworley Are there any Lagoon docs for this?

@nicksantamaria nicksantamaria changed the base branch from 4.x to 5.x November 10, 2022 22:47
@nicksantamaria nicksantamaria force-pushed the build/read-replica-support branch from 3a55336 to e8d5835 Compare November 10, 2022 22:52
@nicksantamaria
Copy link
Contributor

i read up on the read replicas docs for flexible server, and this isn't something thats going to offer us any advantages.

You have to provision read-replicas on a flexible server cluster, which is an extra instance, meaning extra cost. So you might as well increase the resource allocation on the main db cluster and gain benefits for writes too.

@nicksantamaria
Copy link
Contributor

@nicksantamaria nicksantamaria deleted the build/read-replica-support branch October 23, 2023 00:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants