Primary keys#21
Conversation
|
This is a follow up after #19 I recommend not to have a look into it, until the other one is merged. What I have done:
It only works with non-generated PKs, otherwise SQL server causes issues. |
|
@PhenX Ready to be reviewed |
|
|
||
| public string QuotedColumName { get; } = dialect.Quote(property.GetColumnName()); | ||
|
|
||
| public string StoreDefinition { get; } = GetStoreDefinition(property); |
There was a problem hiding this comment.
Not anymore. I tested some stuff with SQL Server and whether it makes more sense to create the temp table manually.
The point is that the identity constraints are somehow copied to the SQL table, therefore you cannot make upserts when you have a auto-incremented ID. I still want to continue experimenting with that.
There was a problem hiding this comment.
It is used now. I create the temp table manually for SQL server now, therefore you can make upserts with auto-incremented IDs now.
|
From my side it is ready to be merged. |
No description provided.