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

check all indexes are present #26

Open
butonic opened this issue Aug 31, 2017 · 4 comments
Open

check all indexes are present #26

butonic opened this issue Aug 31, 2017 · 4 comments

Comments

@butonic
Copy link
Member

butonic commented Aug 31, 2017

just stumbled over a 8.2 postgres instance that did not have pref_userid_appid_key_index. was installed at 8.0. we should somehow detect missing indexes ... or make this a repair step? cc @PVince81

@PVince81
Copy link
Contributor

In general at this point in OC 10 I expect that we'll move all possible tables to migrations. When doing so, we should maybe put the index creation in a separate migration and have a check whether said indices existed before or not.

It is possible that the current migration implementations don't bother with indices if the tables already exist. If that's the case, let's add additional migrations to check and add missing indices.

Repair step => migration is better in this case.

@mrow4a
Copy link
Contributor

mrow4a commented Sep 2, 2017

Hmm it should be possible, you think about a button in the UI?

@butonic
Copy link
Member Author

butonic commented Sep 5, 2017

for mysql:

SELECT DISTINCT TABLE_NAME, INDEX_NAME FROM INFORMATION_SCHEMA.STATISTICS WHERE TABLE_SCHEMA = 'your oc db name'

@butonic
Copy link
Member Author

butonic commented Sep 5, 2017

add this to the config report?

@felixboehm felixboehm added this to the backlog milestone Apr 6, 2018
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

No branches or pull requests

4 participants