-
Notifications
You must be signed in to change notification settings - Fork 96
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
No progress indication if a migration is slow #107
Comments
I would be checking this sometimes later this week but as usual a pull request is always welcomed. |
Hmm, this is trickier than one would expect. Just shoving a // OUTPUT here
printf("========= Begin: %s ======== \n", $file['class']);
$this->_adapter->start_transaction();
$result = $obj->$target_method();
//successfully ran migration, update our version and commit
$this->_migrator_util->resolve_current_version($file['version'], $target_method);
$this->_adapter->commit_transaction(); Functionally works but the display is out of order - since the Task accumulates all output in E.g.
Ideally we would want the |
Is there a specific reason to why the output needs to be accumulated? Why not output everything from there? |
@marcofognog maybe it's just a question of design. Instead of printing in every instance of migration there is a one method in RuckusFramework, which output everything |
In instances where I anticipate slow queries, or major migrations, but http://dev.mysql.com/doc/refman/5.1/en/query-log.html Dmitry Tonkonogov wrote:
|
ruckus.php
does not currently print any output until all migrations have finished.If one or more migrations take a very long time, then this can be disconcerting. It is not clear whether
ruckus.php
has started or not.It would be better if a "starting migration x" message were printed before each migration started.
(Tested on version
d8b1e737
(currentmaster
) on Windows with PHP 5.4.13 + Cygwin shell)The text was updated successfully, but these errors were encountered: