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

Fix schema for Quartz Postgres Store #4864

Merged
merged 1 commit into from
Feb 2, 2024
Merged

Fix schema for Quartz Postgres Store #4864

merged 1 commit into from
Feb 2, 2024

Conversation

mohdali
Copy link
Member

@mohdali mohdali commented Feb 2, 2024

The Postgres Migration for Quartz puts the tables under quartz schema. However, the job store does not have any option to provide schema definition. However, it is possible to specify in the table prefix.

Methods and file names had to be renamed to fix copy error. (might be a breaking change for existing Elsa 3.0).

Copy link
Member

@sfmskywalker sfmskywalker left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @mohdali !

@sfmskywalker sfmskywalker merged commit 109c303 into main Feb 2, 2024
1 of 2 checks passed
@sfmskywalker sfmskywalker deleted the quartz-postgres branch February 2, 2024 16:30
@sfmskywalker sfmskywalker added the elsa 3 This issue is specific to Elsa 3 label Feb 9, 2024
@sfmskywalker sfmskywalker added this to the Elsa 3.1 milestone Feb 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
elsa 3 This issue is specific to Elsa 3
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

2 participants