Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Spurious error log during upgrade #939

Closed
ikedas opened this issue May 12, 2020 · 0 comments · Fixed by #1560
Closed

Spurious error log during upgrade #939

ikedas opened this issue May 12, 2020 · 0 comments · Fixed by #1560
Labels
bug ready A PR is waiting to be merged. Close to be solved
Milestone

Comments

@ikedas
Copy link
Member

ikedas commented May 12, 2020

Version

At least 6.2.40 or later.

Installation method

Any.

Expected behavior

When a database column would be removed during upgrade and it is missing, successful result should be logged.

Actual behavior

Error log stating the column cannot be dropped is shown.

notice Sympa::Upgrade::upgrade() Upgrading admin_table
err main::#890 > Sympa::Upgrade::upgrade#1835 > Sympa::DatabaseDriver::MySQL::delete_field#288 > Sympa::Database::do_query#280 Unable to execute SQL statement "ALTER TABLE admin_table DROP COLUMN date_admin": (42000) Can't DROP COLUMN date_admin; check that it exists
err main::#890 > Sympa::Upgrade::upgrade#1835 > Sympa::DatabaseDriver::MySQL::delete_field#296 Could not delete field date_admin from table admin_table in database sympa
notice Sympa::Upgrade::upgrade() Upgrading subscriber_table
err main::#890 > Sympa::Upgrade::upgrade#1838 > Sympa::DatabaseDriver::MySQL::delete_field#288 > Sympa::Database::do_query#280 Unable to execute SQL statement "ALTER TABLE subscriber_table DROP COLUMN date_subscriber": (42000) Can't DROP COLUMN date_subscriber; check that it exists
err main::#890 > Sympa::Upgrade::upgrade#1838 > Sympa::DatabaseDriver::MySQL::delete_field#296 Could not delete field date_subscriber from table subscriber_table in database sympa

Additional information

@ikedas ikedas added the bug label May 12, 2020
ikedas added a commit to ikedas/sympa that referenced this issue Dec 28, 2022
ikedas added a commit to ikedas/sympa that referenced this issue Dec 29, 2022
@ikedas ikedas added the ready A PR is waiting to be merged. Close to be solved label Jan 4, 2023
@ikedas ikedas added this to the 6.2.72 milestone Feb 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug ready A PR is waiting to be merged. Close to be solved
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant