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

RelationalTypeMapper: FK facet propagation #2455

Closed
AndriySvyryd opened this issue Jun 23, 2015 · 1 comment
Closed

RelationalTypeMapper: FK facet propagation #2455

AndriySvyryd opened this issue Jun 23, 2015 · 1 comment
Labels
closed-fixed The issue has been fixed and is/will be included in the release indicated by the issue milestone. type-enhancement
Milestone

Comments

@AndriySvyryd
Copy link
Member

If the user specifies a store type for a principal key column that is different from the default creating a foreign key might result in a store error due to type mismatch.
This isn't too bad for explicit FKs, as the user can easily configure the store type, but for shadow state FKs and FKs found by convention it would be harder to figure out and would require more code to fix.
RelationalTypeMapper should get the store type from the principal key if it's not set on the foreign key.

@ajcvickers
Copy link
Contributor

@AndriySvyryd Poaching this.

ajcvickers added a commit that referenced this issue Jun 28, 2016
So that if the type mapping for a primary key is changed, then the type mapping for referencing foreign keys will also change.

Issue #2455
@ajcvickers ajcvickers added the closed-fixed The issue has been fixed and is/will be included in the release indicated by the issue milestone. label Jul 6, 2016
@ajcvickers ajcvickers modified the milestones: 1.1.0-preview1, 1.1.0 Oct 15, 2022
@ajcvickers ajcvickers removed their assignment Sep 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
closed-fixed The issue has been fixed and is/will be included in the release indicated by the issue milestone. type-enhancement
Projects
None yet
Development

No branches or pull requests

3 participants