Skip to content

Start of considering unique and not null contraints.#65

Draft
hydrogen602 wants to merge 4 commits intoKarnerTh:masterfrom
hydrogen602:unique-constraint-start
Draft

Start of considering unique and not null contraints.#65
hydrogen602 wants to merge 4 commits intoKarnerTh:masterfrom
hydrogen602:unique-constraint-start

Conversation

@hydrogen602
Copy link

This PR is a possible start to adding 2 new relations:

  • relationOneToMaybeOne: |o--o|
  • relationManyToMaybeOne: }o--o|

The code makes use of the IsUnique and IsNullable properties of ColumnResult, however, in its current form, it can't differentiate between one column being unique vs multiple columns being unique (IsUnique is true for both).

@KarnerTh KarnerTh self-requested a review January 1, 2025 17:10
@hydrogen602
Copy link
Author

As far as I can see, differentiating between individual unique columns vs multiple columns being unique together would require collecting additional info from the database and so would be a bigger change.

@KarnerTh KarnerTh force-pushed the master branch 4 times, most recently from 0884560 to 76c5d90 Compare February 12, 2025 08:17
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.

2 participants