Re: [PATCH] pgarchives: Bugfix: missing ids in pglister_sync
| От | Célestin Matte |
|---|---|
| Тема | Re: [PATCH] pgarchives: Bugfix: missing ids in pglister_sync |
| Дата | |
| Msg-id | a5b19a46-2069-f3ec-d177-35aa46c12a8f@cmatte.me обсуждение исходный текст |
| Ответ на | Re: [PATCH] pgarchives: Bugfix: missing ids in pglister_sync (Célestin Matte <celestin.matte@cmatte.me>) |
| Ответы |
Re: [PATCH] pgarchives: Bugfix: missing ids in pglister_sync
|
| Список | pgsql-www |
>> I think the attached patch is more correct (I've backpatched it into >> the 0001 migration because making it a separate migration makes it >> somewhat destructive -- it's easier to just change it manually on >> existing installs if you have one) > > Thanks! Can it be pushed into the git repository, or does it need testing or something? > I understand that changing id fields to AutoFields can be destructive for existing install, but on the other hand, the currentstate of the repository is problematic as it causes pglister_sync to fail by default. Another solution could be to add sequences directly in postgres using RunSQL() in a migration to turn these fields into defacto serial fields. This is a bit hacky, but non-destructive. What do you think? -- Célestin Matte
В списке pgsql-www по дате отправления: