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 4 days ago by ced.

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
2020-10-22 00:05:04cedsetstatus: chatting -> testing
2020-10-19 19:07:25reviewbotsetmessages: + msg61060
2020-10-09 10:51:26reviewbotsetmessages: + msg60697
2020-09-29 10:59:57reviewbotsetmessages: + msg60461
2020-09-28 23:10:55nicoesetmessages: + msg60456
2020-09-28 18:28:53reviewbotsetnosy: + reviewbot
messages: + msg60452
2020-09-28 18:28:53reviewbotsetreviews: 312421002
keyword: + review
2020-09-28 18:14:34pokolisetstatus: unread -> chatting
2020-09-28 18:14:26pokolisetnosy: + ced
2020-09-28 18:14:13pokolisetstatus: chatting -> unread
nosy: + pokoli, - ced

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