-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Use all-caps table names in the liquibase script #340
Use all-caps table names in the liquibase script #340
Conversation
Hi @immortaly007 Thank you for the PR! Would you mind providing some instruction as how to use this file? You can post them here as comment, and I will try to create a WIKI page so other can use it. |
Hi @zemian, The file is useful for users that want to control their database schema using Liquibase, as opposed to using the "quartz standard" way of generating/controlling the database schema. The advantage of using Liquibase is that this might integrate better with the existing schema generation, and it keeps the application and quartz schema generation in one place. The easiest way the file can be used is to copy it next to the users' existing changelog file (for example to
This way, Liquibase will ensure that the quartz tables are created (with their default names). Note: one could probably also reference the file directly on the quartz classpath, but I'm not sure what that path would be, and copying the file allows the user to modify it if needed. When using the
|
Thank you @immortaly007 for the info! Few updates I have for this PR:
|
Use all-caps table names in the liquibase script #340
Use all-caps table names in the liquibase script #340
On case-sensitive databases, the table names in the Liquibase script cause issues, since they're all lower case. This change fixes that.
It also removes the
tdss
prefix from the table names.