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

[wip] Add MongoDB support #5

Open
wants to merge 14 commits into
base: master
Choose a base branch
from
Open

[wip] Add MongoDB support #5

wants to merge 14 commits into from

Conversation

gravis
Copy link
Member

@gravis gravis commented Aug 15, 2016

taken from mattes/migrate#118

dimag-jfrog and others added 13 commits August 4, 2016 15:52
- Added go methods migrator, mongo db template: different from the usual driver
model.
- Added support for bidirectional files (for go methods), appending _up or _down upon context
- Added DriverWithFilnameParser for providing custom filename parser functionality that knows to parse bi-directional file names.
- Changed logic not to use custom filename parser:
  -Supporting up and down files only, no both direction files
  -Using method names as is
- Added complete test to the mongo db migration scenarios
…cific methods

- defined methods receivers registration by name and change the migration files format to include them
- added extensive testing
- currently each driver contains only a single method receiver:
   - enforcing method receiver pre-registration on go methods driver initialization
   - Method receiver name can be removed from files format
- passing the DbName parameter inside the method receiver for the go methods driver
@heynemann
Copy link

Any idea what's required to get this approved?

@gravis
Copy link
Member Author

gravis commented Sep 22, 2016

This: mattes/migrate#118 (comment)

@gravis
Copy link
Member Author

gravis commented Sep 22, 2016

I understand it's hard to follow, I wish mattes/migrate wasn't dead too :(

@gravis
Copy link
Member Author

gravis commented Nov 22, 2016

Any news on this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants