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

PostgreSQL SQL Migration Documentation Issue #114

Closed
esetnik opened this issue Mar 26, 2017 · 5 comments · Fixed by #128
Closed

PostgreSQL SQL Migration Documentation Issue #114

esetnik opened this issue Mar 26, 2017 · 5 comments · Fixed by #128
Milestone

Comments

@esetnik
Copy link

esetnik commented Mar 26, 2017

The table_postgres.sql should use DROP TABLE IF EXISTS syntax so that initial migration can be run on a new database.

DROP TABLE IF EXISTS qrtz_fired_triggers;
DROP TABLE IF EXISTS QRTZ_PAUSED_TRIGGER_GRPS;
DROP TABLE IF EXISTS QRTZ_SCHEDULER_STATE;
DROP TABLE IF EXISTS QRTZ_LOCKS;
DROP TABLE IF EXISTS qrtz_simple_triggers;
DROP TABLE IF EXISTS qrtz_cron_triggers;
DROP TABLE IF EXISTS qrtz_simprop_triggers;
DROP TABLE IF EXISTS QRTZ_BLOB_TRIGGERS;
DROP TABLE IF EXISTS qrtz_triggers;
DROP TABLE IF EXISTS qrtz_job_details;
DROP TABLE IF EXISTS qrtz_calendars;
@jhouserizer
Copy link
Contributor

fine suggestion, thanks.

jhouserizer added a commit that referenced this issue Apr 16, 2017
…resql

resolve #114  - add 'if exists' clause to drop tables command on postgresql tables
@esetnik
Copy link
Author

esetnik commented Apr 18, 2017

Sorry about not submitting a PR. I didn't realize these scripts were contained in the repo. Thanks for the fix @jhouserizer.

@Ramblurr
Copy link

Ramblurr commented May 8, 2017

I think there was a typo in that commit: https://github.com/quartz-scheduler/quartz/pull/128/files#r115279504

@esetnik
Copy link
Author

esetnik commented May 10, 2017

good catch @Ramblurr looks like it's a typo

@zemian zemian modified the milestone: 2.3.0 Jul 1, 2017
@rraziel
Copy link

rraziel commented Aug 24, 2018

Hello, that typo noticed by @Ramblurr went into the 2.3.0 release.

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 a pull request may close this issue.

5 participants