Bug #4871: Migration SQL script to change the format of the ids from integer to bigint - Rudder - Issue Tracker

Actions

Copy link

Bug #4871

closed

Bug #4831: Reports cannot be inserted into postgresql due to a limit on ids

Migration SQL script to change the format of the ids from integer to bigint

Added by Nicolas CHARLES over 10 years ago. Updated over 10 years ago.

Status:

Released

Priority:

1 (highest)

Assignee:

François ARMAND

Category:

Web - Compliance & node report

Target version:

2.10.1

Pull Request:

https://github.com/Normation/rudder/p...

Severity:

UX impact:

User visibility:

Effort required:

Priority:

Name check:

Fix check:

Regression:

Description

This ticket is to create the SQL migration script to :
- rename the tables ruddersysevent to ruddersysevents_old, and create a new ruddersysevetns with bigint ids
- rename the tables archivedruddersysevent to archivedruddersysevents_old, and create a new archivedruddersysevetns with bigint ids

And another script to copy a specific number of reports from a node from the ruddersysevents_old to ruddersysevent

Actions

Copy link

Updated by Nicolas CHARLES over 10 years ago

Status changed from In progress to Pending technical review
Assignee changed from Ade Bradshaw to François ARMAND
Pull Request set to https://github.com/Normation/rudder/pull/517

https://github.com/Normation/rudder/pull/517

Actions

Copy link

Updated by Nicolas CHARLES over 10 years ago

Tracker changed from Enhancement to Bug

Actions

Copy link

Updated by Christophe Nowicki over 10 years ago

Hi Nicolas,

Maybe bigserial is better than bigint?

http://docs.postgresql.fr/8.1/datatype.html#datatype-serial

My 0.02 BTC

Actions

Copy link

Updated by Nicolas CHARLES over 10 years ago

Oh, good catch Christophe.

Actions

Copy link

Updated by Nicolas CHARLES over 10 years ago

Actually, there is no need to migrate the sequence:

rudder=> \d serial
           Sequence "public.serial" 
    Column     |  Type   |        Value        
---------------+---------+---------------------
 sequence_name | name    | serial
 last_value    | bigint  | 14566345
 start_value   | bigint  | 101
 increment_by  | bigint  | 1
 max_value     | bigint  | 9223372036854775807
 min_value     | bigint  | 1
 cache_value   | bigint  | 1
 log_cnt       | bigint  | 19
 is_cycled     | boolean | f
 is_called     | boolean | t

http://www.postgresql.org/docs/8.1/static/sql-createsequence.html
For once, default are sane in postgresql :)

Actions

Copy link