Issue 9645

Title
Payment term's PaymentTermLineRelativeDelta does too many migration
Priority
bug
Status
testing
Nosy list
ced, jcavallo, nicoe, pokoli, reviewbot
Assigned to
nicoe
Keywords
PostgreSQL, review

Created on 2020-09-28.18:10:25 by nicoe, last changed 3 weeks ago by reviewbot.

Messages

Author: [hidden] (nicoe) Tryton committer
Date: 2020-09-28.23:10:55
* Cédric Krier [septembre 28, 2020 18:14]:
> 
> Cédric Krier <cedric.krier@b2ck.com> added the comment:
> 
> I think it is too complex to manage the column type.
> What about testing by inserting a row with the proper type?

Unfortunately it does not work because postgres is doing an implicit
CAST when inserting a value into a VARCHAR and it's bypassable only by
fiddling with the catalog stuffs.
-- 
Nicolas Évrard - B2CK SPRL
E-mail/Jabber: nicolas.evrard@b2ck.com
Tel: +32 472 54 46 59
Website: http://www.b2ck.com/
Author: [hidden] (ced) Tryton committer Tryton translator
Date: 2020-09-28.18:14:05
I think it is too complex to manage the column type.
What about testing by inserting a row with the proper type?
Author: [hidden] (nicoe) Tryton committer
Date: 2020-09-28.18:10:25
In order to fix issue7962 we introduced a change that drop and recreate the columns month and weekday even if there is no migration to handle.

When executing the migration a great number of time we reach the maximum number of column permitted on a postgresql because postgres never removes the dropped column but hides it: https://nerderati.com/2017/01/03/postgresql-tables-can-have-at-most-1600-columns/

I propose to add to the backend table handler a way to retrieve the column type and change the test to check if the column should be dropped or not.
History
Date User Action Args
2021-01-06 14:14:33reviewbotsetmessages: + msg63792
2021-01-05 16:48:48reviewbotsetmessages: + msg63768
2020-12-16 11:47:08reviewbotsetmessages: + msg63298
2020-12-11 16:26:13reviewbotsetmessages: + msg63036
2020-12-01 13:59:35reviewbotsetmessages: + msg62325
2020-12-01 00:29:17reviewbotsetmessages: + msg62324
2020-11-20 14:25:41reviewbotsetmessages: + msg62058
2020-10-22 00:05:04cedsetstatus: chatting -> testing
2020-10-19 19:07:25reviewbotsetmessages: + msg61060
2020-10-09 10:51:26reviewbotsetmessages: + msg60697

Showing 10 items. Show all history (warning: this could be VERY long)