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.

Sign up for GitHub

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

Jump to bottom

Better support for foreignKey elements #103

Merged
merged 7 commits into from Sep 15, 2015
Merged

Better support for foreignKey elements #103

merged 7 commits into from Sep 15, 2015

Conversation

austinhyde
Copy link
Member

This should resolve #88 and #100:

  • Generates missing foreign key indexes in MySQL
  • Extracts compound foreign keys as foreignKey elements
  • Adds onUpdate/onDelete attributes for referential constraints
  • Fixes bug where indexName wasn't getting relayed correctly

@austinhyde austinhyde added this to the v1.4.1 milestone Sep 12, 2015
@nkiraly nkiraly self-assigned this Sep 15, 2015
nkiraly added a commit that referenced this pull request Sep 15, 2015
@nkiraly
Merge pull request #103 from austinhyde/mysql-magic-compound-indexes
7bbfd97 
Better support for foreignKey elements
@nkiraly nkiraly merged commit 7bbfd97 into dbsteward:master Sep 15, 2015
@nkiraly nkiraly mentioned this pull request Sep 15, 2015
Closed
nkiraly added a commit that referenced this pull request Sep 15, 2015
@nkiraly
add PR #103 fixes for foreign keys
930363c 
mysql5 foreign key index extraction and handling improvements github/nkiraly/DBSteward PR #103 issue #88
foreign key definition additions for ON DELETE, ON UPDATE github/nkiraly/DBSteward PR #103 issue #100
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@nkiraly nkiraly

Labels
extraction improvement mysql pgsql sql-output xml-parsing
Projects
None yet
Milestone
v1.4.1
Development

Successfully merging this pull request may close these issues.

mysql driver does not handle composite foreign key "magic indexes"
2 participants
@austinhyde @nkiraly