Proposed changes for indexing.#3347
Proposed changes for indexing.#3347rohit-armorikar wants to merge 4 commits intocloudfoundry:developfrom
Conversation
Adding mysql proposed indexing change.
Adding file for postgres.
Adding file for hsqldb
|
@rohit-armorikar Thank you! We will check, please sign CLA as required step to accept your PR, thanks in advance |
|
ok, you can and should run tests before the PR, see unit test failing because of index org.springframework.beans.BeanInstantiationException: Failed to instantiate [org.flywaydb.core.Flyway]: Factory method 'flyway' threw exception; nested exception is org.flywaydb.core.internal.command.DbMigrate$FlywayMigrateException: Migration V4_113__Add_Index_user_group_membership_groups.sql failed We run on MYSQL 5 and 8 and both fail. With docker and this help you can run tests yourself. |
Corrected the issues with queries as per MySql 8 version.
|
@strehle Thanks for your input, we have corrected the changes and pushed the updates. |
We have added few proposal for required indexing on tables.
Please review and let us know, which index out of the proposed one will help us for performance improvement.