Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

enh(73): Add index to NEWS_CONFIGURATION.SET_ID. #74

Merged
merged 1 commit into from
Aug 29, 2018
Merged

enh(73): Add index to NEWS_CONFIGURATION.SET_ID. #74

merged 1 commit into from
Aug 29, 2018

Conversation

bjagg
Copy link
Member

@bjagg bjagg commented Aug 27, 2018

Fixes #73.

@bjagg bjagg added enhancement WIP Work in progress labels Aug 27, 2018
@bjagg
Copy link
Member Author

bjagg commented Aug 27, 2018

Still needs to be tested with databases.

HSQL: Hibernate creates 2 indices

@bjagg
Copy link
Member Author

bjagg commented Aug 29, 2018 •
edited
Loading

HSQL: adds a duplicate index
MySQL: adds a duplicate index
Postgres: adds missing index
MS SQL: adds missing index
Oracle: adds missing index

@drewwills
Copy link
Contributor

What's up with duplicate indices?

@bjagg
Copy link
Member Author

bjagg commented Aug 29, 2018

HSQL and MySQL will automatically create an index on foreign keys while the other databases do not.

@bjagg bjagg merged commit 28d400a into uPortal-Project:master Aug 29, 2018
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
enhancement WIP Work in progress
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@bjagg @drewwills