Project

General

Profile

Actions

Bug #2684

closed

The fileFormat migration script for Rudder 2.4 (version 1) contains a flawed condition

Added by Matthieu CERDA over 12 years ago. Updated almost 10 years ago.

Status:
Released
Priority:
1 (highest)
Assignee:
Matthieu CERDA
Category:
Packaging
Target version:
Severity:
UX impact:
User visibility:
Effort required:
Priority:
Name check:
Fix check:
Regression:

Description

The script should be:

select count(*) from (
select xpath('/entry/*[@fileFormat=1]',data) AS x
from eventlog) as Y
where array_upper(x, 1) > 0;

and

select count(*) from (
select xpath('/entry/addPending',data) AS x
from eventlog) as Y
where array_upper(x, 1) > 0;

But on the rudder-upgrade script only the second test was inserted twice.


Related issues 1 (0 open1 closed)

Related to Rudder - Architecture #2364: Test for the XML fileformat in migration script for EventLog migration in DBReleasedMatthieu CERDA2012-03-06Actions
Actions #1

Updated by Matthieu CERDA over 12 years ago

  • Status changed from In progress to Pending technical review
  • % Done changed from 0 to 100

Applied in changeset commit:cb5a7a780f2afb1575dcd44439634fc0bac43480.

Actions #2

Updated by Nicolas CHARLES over 12 years ago

  • Status changed from Pending technical review to Released

This looks valid, thank you Matthieu

Actions #3

Updated by Nicolas PERRON almost 12 years ago

  • Project changed from Rudder to 34
  • Category deleted (11)
Actions #4

Updated by Benoît PECCATTE almost 10 years ago

  • Project changed from 34 to Rudder
  • Category set to Packaging
Actions

Also available in: Atom PDF