search

department-of-veterans-affairs / veterans-employment-center

Veterans Employment Center
https://www.vets.gov/veterans-employment-center
4 stars 4 forks source link

Add index on skills is_common #374

Closed patrickvinograd closed 7 years ago

patrickvinograd commented 7 years ago

Fixes issue observed in production: https://github.com/department-of-veterans-affairs/devops/issues/873

We have not seen this VEC issue re-occur, and there is little harm in adding a column index. The skills table is is used by resume-builder and so will live on for a while at least.

Verified that db:migrate is happy even if the index by the same name already exists, as is the case in production.

jkassemi commented 7 years ago

Looks like there may be some issues with CI that need addressed, pretty sure those aren't related to these changes.