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 Indices Related Operation at Migration #99

Merged
merged 5 commits into from
Feb 19, 2014

Conversation

fudanchii
Copy link

Keep indices intact when columns renamed.
Tests included.

As discussed in #83, this is using:

select sql from sqlite_master where type = 'index' and tbl_name = ?

instead of PRAGMA stuff for fetching index's ddl.

Thanks to @bradrydzewski for the hint 👍

I also add migration script generator to ensure we have incremental Revision number.

I also want to notice that the AddColumn method is still translated directly to SQLite ALTER TABLE which means we still have the limitations (e.g. can't add column with PRIMARY KEY and/or UNIQUE constraints), I think this is worth another discussion, and eventually this may turn into whether we want to stay and fix these or move to another rdbms, but I wont fuss about it right now.

Thanks

bradrydzewski added a commit that referenced this pull request Feb 19, 2014
Fix Indices Related Operation at Migration
@bradrydzewski bradrydzewski merged commit 30a9845 into harness:master Feb 19, 2014
@fudanchii fudanchii deleted the migration branch March 8, 2014 05:36
johannesHarness pushed a commit that referenced this pull request Sep 26, 2023
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.

None yet

2 participants