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

Import all indexes including their names. #5

Merged
merged 1 commit into from
May 12, 2015
Merged

Conversation

DouweM
Copy link

@DouweM DouweM commented May 9, 2015

@dosire
Copy link
Member

dosire commented May 10, 2015

@DouweM awesome! I assume you'll make sure to update the docs when this is merged

@DouweM
Copy link
Author

DouweM commented May 10, 2015

@dosire That was the idea :)

@jacobvosmaer
Copy link

Awesome @DouweM . Could you paste the results of the test procedure outlined in the other PR?

The second step after fixing the converter would be to write a (one-time) migration in gitlabhq to rename existing different-named indexes.

jacobvosmaer added a commit that referenced this pull request May 12, 2015
Import all indexes including their names.
@jacobvosmaer jacobvosmaer merged commit c3d295a into gitlab May 12, 2015
@DouweM DouweM deleted the index-names branch May 12, 2015 11:28
@jacobvosmaer
Copy link

Created #6, #7 to eliminate the schema diff.

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

Successfully merging this pull request may close these issues.

Unique indexes are named different after the conversion
3 participants