Inserts fail for many tables here / Proposal to use SERIAL instead.

Question

Inserts fail for many tables here / Proposal to use SERIAL instead.

Opened this issue 7 months ago · 3 comments

valhuber commented 7 months ago

Thanks again for preparing this.

I use it for testing, and encountered some issues on auto generated keys.

This page describes that:

SERIAL fields work well, and
can be initialized to support pre-loading of data.

These resulted my changes to your ddl, as noted in one of the links.

Answer 1 · 2024-04-25T10:16:59.000Z

Hello,

Thanks for mentioning this.

I do not work much anymore on databases (and I'm sorry for not answering your proposition from one or two years ago about it).

Could you please show me a diff of what changes would be required maybe via pull request. Also, are you sure that the serial type works with older versions of Postgresql (i.e. those that are 3 years old or less).

Answer 2 · 2024-04-25T11:34:41.000Z

Your ddl is in that same folder if you follow the link. Not sure about versions, maybe just use both? Thanks, Val 510-703-5909 valhuber on skype

…

On Thu, Apr 25, 2024 at 3:17 AM Pascal Thomet ***@***.***> wrote: Hello, Thanks for mentioning this. I do not work much anymore on databases (and I'm sorry for not answering your proposition from one or two years ago about it). Could you please show me a diff of what changes would be required maybe via pull request. Also, are you sure that the serial type works with older versions of Postgresql (i.e. those that are 3 years old or less). — Reply to this email directly, view it on GitHub <#19 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ABKSG32LWXZZT2B3RRD4G5DY7DJ3BAVCNFSM6AAAAABGXXV57OVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANZWHA2DKNBTGE> . You are receiving this because you authored the thread.Message ID: ***@***.***>

Answer 3 · 2024-04-25T12:43:53.000Z

I will leave this issue open so that user can find it if needed.