Currently when you call InsertAsync (for example) and the PK is non identity column (i.e UUID with default) or a PK is already provided by the entity you are inserting, the Insert method returns an empty PK value.
This change would make it so you are ALWAYS provided the PK of the entity from the Insert even if you provided it yourself or the DB generated it outside of an identity situation, this provides a more consistent experience for the library consumer. It may also be worth noting that the PK is only returned if it is an identity field run on the DB.
This was discussed in a gitter conversation with @mikependon and he was happy for it to be raised here as a request.
Currently when you call
InsertAsync(for example) and the PK is non identity column (i.e UUID with default) or a PK is already provided by the entity you are inserting, theInsertmethod returns an empty PK value.This change would make it so you are ALWAYS provided the PK of the entity from the
Inserteven if you provided it yourself or the DB generated it outside of anidentitysituation, this provides a more consistent experience for the library consumer. It may also be worth noting that the PK is only returned if it is an identity field run on the DB.