Convert datid to a string for backwards compatibility

[danielnelson]

closes #3622

Required for all PRs:

• Signed CLA.
• Associated README.md updated.
• Has appropriate unit tests.

[danielnelson]

@james-lawrence I could use some advice here if you can spare the time. Please ignore the code here so far, it won't work.

I introduced a regression into 1.5 causing oid types to be converted to ints instead of strings, so I'm trying to undo that. Since this also affects postgresql_extensible I would like it to apply to all oids.

It seems like to fix this I will need to pass the row.ColumnTypes() into accRow() and then use that to setup the columnMap and columnVars with the correct types. Do you think this is the best way to go about a fix?

[james-lawrence]

@danielnelson do you know what commit caused the regression so I can take a look?

[danielnelson]

It was part of #3210, when we updated pgx. I believe it was added in jackc/pgx@bec9bd2, quite some time back.

[james-lawrence]

ah, so basically the driver added first class support for the type which changed the default output.

unfortunately atm there isn't a good way of doing this. once jackc/pgx#365 is merged we should be able to configure the output for types.

[danielnelson]

Too bad, I guess I'll just whip a release note for this item then. Thanks for the help