Ticket #1151 (closed Bug: fixed)

Opened 2 years ago

Last modified 2 years ago

Make it very clear when new versions have large schema updates

Reported by: matt Owned by:
Priority: critical Milestone: Piwik 0.5.5
Component: Core Keywords:
Cc: Sensitive: no

Description

We had issues in the past (in 0.5) when we released large schema upgrade causing time outs on some hosts, and we had to offer a lot of support in the forums.

I would like to propose a few steps to make this better in future releases (0.5.5 looks like it might contain such massive schema upgrade)

1) Improve the messaging in the welcome update page and display a big red warning message when a large schema update will happen. This is the case whenever a large table (log_ or archive_) have new fields, new INDEXes, or various other bulk updates. We could have a flag/method in the update class that would be set to true when the update can potentially lead to such issues.

2) We should also display the list of SQL queries that are going to be executed so that the user can execute them manually in case of error.

3) Since 0.5 there is a feature that updates can be ran using the safer command line upgrade mechanism. This should also be highlighted.

Change History

Changed 2 years ago by vipsoft

(In [1844]) refs #1129, refs #1151 - refactoring to add getSql() to return array of SQL statements

Changed 2 years ago by matt

Note: I created the ticket after reading the post mortem at:  http://devzone.zend.com/article/11821-Yes-I-Crashed-the-Site

Changed 2 years ago by vipsoft

Brain dump. Not sure how practical this idea is...

see:  http://www.php.net/manual/en/function.exec.php#86329

Idea is: (pseudocode)

if not php cli mode
    ignore_user_abort
    disable max_execution_time
        
    if exec() is not disabled
        exec('nohup php index.php >/dev/null 2>/dev/null &') // run the update in the background
        redirect to tmp/latest/progress.html // a static file that is overwritten by the updater
    else
        run updater // for each update, echo 'something' to the client and flush output buffers to (try) keep connection alive

else
    run updater // foreach update, write to progress.html

Changed 2 years ago by matt

Anthon, I'm not going to test your pseudocode, it should like it might work, but generally if users have the exec() command, they might just want to run the command line updater I think.

let me know what you think of my improvements to the update welcome page; I hope this will help users get through large updates in the future

Changed 2 years ago by matt

in [1941] (again trac didn't pick up this commit, grr)

Fixes #1151 Clarifying update process for large piwik instances. outputs list of SQL queries that will be executed during updates. Fixes language selector during update, which was only working after second refresh.

Changed 2 years ago by matt

  • status changed from new to closed
  • resolution set to fixed
Note: See TracTickets for help on using tickets.