Update lib/Doctrine/Common/DataFixtures/Executor/MongoDBExecutor.php #72

Closed
wants to merge 1 commit into
from

4 participants

@weyandch

When loading fixtures with MongoDB the indexes are getting dropped on purging the collections.

Added the proper command to update the Indexes right after the purging, to ensure consistency

@weyandch weyandch Update lib/Doctrine/Common/DataFixtures/Executor/MongoDBExecutor.php
When loading fixtures with MongoDB the indexes are getting dropped on purging the collections.

Added the proper command to update the Indexes right after the purging, to ensure consistency
6e12d48
@jmikola
Doctrine member

It may be a good idea to hold off on this until we have a tag in ODM next week, since this was a relatively new feature.

@stof
Doctrine member

And btw, shouldn't it be done by the purger instead of the executor ?

@weyandch

i judged the purger as the tool to wipe all the data, thus i didn't want to mix in a task which generates something at the same time..
would it be an option to control the execution by an optional command line argument... like --drop-index / --restore-index, think restoring should be enabled by default, as it ensures consistency of your data.. but leaves the option to get rid of 'em if necessary

@lavoiesl
Doctrine member

Closed in favour of #118

@lavoiesl lavoiesl closed this Mar 23, 2015
@lavoiesl lavoiesl added the Duplicate label Mar 23, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment