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.

Sign up for GitHub

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

Jump to bottom

Add missing index to original migration #19761

Merged
merged 2 commits into from
Apr 6, 2020
Merged

Add missing index to original migration #19761

merged 2 commits into from
Apr 6, 2020

Conversation

georgehrke
Copy link
Member

fixes #19658

@georgehrke
Add indices to original DAV migration
7961a0b 
Signed-off-by: Georg Ehrke <developer@georgehrke.com>
@georgehrke georgehrke added the 2. developing Work in progress label Mar 3, 2020
@georgehrke georgehrke force-pushed the bugfix/19658/add_missing_index_to_original_migration branch from f47caf1 to 31bb58f Compare March 3, 2020 12:49
@georgehrke
Create Mount IDs as BigInt by default
79dcd34 
Signed-off-by: Georg Ehrke <developer@georgehrke.com>
@georgehrke georgehrke force-pushed the bugfix/19658/add_missing_index_to_original_migration branch from 31bb58f to 79dcd34 Compare March 3, 2020 13:17
@rullzer rullzer added this to the Nextcloud 19 milestone Apr 4, 2020
@rullzer rullzer added 3. to review Waiting for reviews and removed 2. developing Work in progress labels Apr 4, 2020
@rullzer rullzer mentioned this pull request Apr 4, 2020
Merged
80 tasks
nickvergessen
nickvergessen approved these changes Apr 4, 2020
View reviewed changes
Copy link
Member

@nickvergessen nickvergessen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Didn't check with the follow ups, but looks good

@rullzer rullzer merged commit 9a4193f into master Apr 6, 2020
@rullzer rullzer deleted the bugfix/19658/add_missing_index_to_original_migration branch April 6, 2020 07:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nickvergessen nickvergessen nickvergessen approved these changes

@rullzer rullzer rullzer approved these changes

@juliusknorr juliusknorr Awaiting requested review from juliusknorr

@ChristophWurst ChristophWurst Awaiting requested review from ChristophWurst

Assignees
No one assigned
Labels
3. to review Waiting for reviews
Projects
None yet
Milestone
Nextcloud 19
Development

Successfully merging this pull request may close these issues.

Fresh Nextcloud 18.0.1 installation reports missing indexes and that some columns are not set to big int
3 participants
@georgehrke @rullzer @nickvergessen