Update PostgresAdapter to Generate Id Column with AppendColumnName

[dogac00]

Currently, PostgresAdapter generates query for columns with quotes like this: "Column". But it does not add quotes in returning clause when we are returning Id.

I think this PR should fix "Column Id does not exist." error we encounter in PostgreSQL while using InsertAsync.

Related issues:
#481
https://github.com/StackExchange/Dapper/issues/1213
https://github.com/StackExchange/Dapper/issues/1541

[NickCraver]

Unfortunately, this is a breaking change. While it may help you, it will break others - please see #722 on future plans there. In the mean time, you can quote the column name in your attributes if you want it to be case-sensitive.

[dogac00]

Which attribute can we use on our property if we want it to be case-sensitive and quoted? As far as I know, Dapper.Contrib currently does not have ColumnAttribute.

Also even if we add ColumnAttribute on our Id it will still generate RETURNING Id from this line sb.Add(property.Name) and give error in PostgreSQL because my column name there is "Id".

I think we have to implement this breaking change if we want to support Column name mapping.

[dogac00]

In addition, don't you think there is an inconsistency in this generated query:

INSERT INTO Products ("Name", "ItemNumber") VALUES (@Name, @ItemNumber) RETURNING Id

Columns are quoted but Id is not. I think if we quote columns we should quote Id, because it is a column too.

[NickCraver]

I can't speak to Contrib's future plans here and if this is a good fit, but please see #1658 - Dapper.Contrib is not located at https://github.com/DapperLib/Dapper.Contrib (separate from this repository).

Note: I based the Dapper.Contrib fork on the original codebase which hopefully makes porting pull requests against the main branch there much easier - the intent was to split, not throw away work the community wants to go with.