← Back to team overview

credativ team mailing list archive

[Bug 953468] Re: rename_columns should have extra checks

 

To be a little more clear about what I propose:

- pre- or post scripts may have an overall check whether they apply to
the current state of the database. This makes a script robust against
being called multiple times, so that it can simply exit while logging a
warning.

- if rename_columns detects a database layout inconsistent with its
arguments, this should raise a fatal error.

-- 
You received this bug notification because you are a member of
OpenUpgrade Committers, which is the registrant for OpenUpgrade Server.
https://bugs.launchpad.net/bugs/953468

Title:
  rename_columns should have extra checks

Status in OpenUpgrade Server:
  Confirmed

Bug description:
  - check that table_exists
  - check that target column does not exist
  - check that source column exist

  and return:
   True if the rename is actually performed
   False otherwise

  Maybe some added verbose logging.

To manage notifications about this bug go to:
https://bugs.launchpad.net/openupgrade-server/+bug/953468/+subscriptions


References