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

Database version backups #1000

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Database version backups #1000

wants to merge 1 commit into from

Conversation

MattWhelan
Copy link
Contributor

@MattWhelan MattWhelan commented Sep 6, 2024

We use the database version string to detect the product and version of the database (and thus the SQL dialect, among other things). That happens at connection time, so we need to restore the version from the memo if the target DB is down.

I made a few tweaks to the workload tools to test this with Oracle.


This change is Reviewable

@MattWhelan
Copy link
Contributor Author

This is a pretty small change. As I'm heading out on PTO for two weeks, feel free to may any changes you see fit to make in my absence.

There isn't an automated integration test. I had to manually run through the process of setting up an initial changefeed so that the schema/version would be cached, and then using docker network disconnect, pull the plug on the database, to test the disconnected case. Driving all that within a CI/CD test seems... daunting.

internal/util/stdpool/my.go Fixed Show fixed Hide fixed
This allows replicator to start without the database.

Added a shell script to run precommit checks at qol/fmt.sh.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant