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
PSL: Implementing map
field for @default attribute on SQL Server
#2156
Conversation
cbee5e3
to
7be2e7b
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Last comment is the important one. Depending on how much time we have, I think we should maybe look at attributes.rs
and course correct.
} | ||
|
||
db_name.map(|cow| cow.to_string()) | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I already had concerns and re-read the code for constraint names validation in attributes.rs
, now I'm certain we should change the approach asap, we're going down the wrong path (this code does not belong here). Do you have time for a call today so we can sketch the better approach? I'd also be fine merging this and correcting later, but it's only getting more expensive to fix.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We'll have a call later. Let's do this!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
On a call, we as a team decided to redo this after GA.
introspection-engine/connectors/sql-introspection-connector/src/introspection_helpers.rs
Outdated
Show resolved
Hide resolved
…2156) * Make `DefaultValue` a struct * Actually migrate named defaults * Start testing * More tests * Rest of the test, ME/IE + PSL lower * Make `DefaultValue` properties private. * clippy * clippy * Remove DRY parts * Fix error string * Use db_names to define the default constraint name * clippy
Closes: prisma/prisma#8158