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

Improve compatibility with Aiven DB #1413

Merged
merged 1 commit into from Mar 26, 2021

Conversation

wxing1292
Copy link
Contributor

@wxing1292 wxing1292 commented Mar 26, 2021

What changed?

  • Try using a set of default database names when initializing admin DB connection

Why?
Aiven DB is a PostgreSQL compatible DB, but comes with a different default DB name
Close #1389

How did you test it?
Existing test

Potential risks
N/A

Is hotfix candidate?
No

* Try using a set of default database names when initializing admin connection

Aiven DB is a PostgreSQL compatible DB, but comes with a different default DB name
@wxing1292 wxing1292 requested review from alexshtin and a team March 26, 2021 23:28
@wxing1292 wxing1292 enabled auto-merge (squash) March 26, 2021 23:29
@wxing1292 wxing1292 merged commit c34d9bb into temporalio:master Mar 26, 2021
wxing1292 added a commit that referenced this pull request Mar 29, 2021
* Try using a set of default database names when initializing admin connection

Aiven DB is a PostgreSQL compatible DB, but comes with a different default DB name
@wxing1292 wxing1292 deleted the postgresql-compatibility branch April 26, 2021 23:29
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.

temporal-sql-tool default postgres database
2 participants