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

🐛 add guard clauses to add_indices_to_bulkrax migration #862

Merged
merged 1 commit into from
Oct 2, 2023

Conversation

ShanaLMoore
Copy link
Contributor

@ShanaLMoore ShanaLMoore commented Oct 2, 2023

Add guard clauses to add_indices_to_bulkrax migration:

Not doing so produced the following error in Adventist, when trying to run migrations: ArgumentError: Index name 'index_bulkrax_entries_on_identifier' on table 'bulkrax_entries' already exists

Not doing so produced the following error in Adventist, when trying to run migrations: "ArgumentError: Index name 'index_bulkrax_entries_on_identifier' on table 'bulkrax_entries' already exists"
@ShanaLMoore ShanaLMoore added the minor-ver for release notes label Oct 2, 2023
@ShanaLMoore ShanaLMoore merged commit 1fc65e4 into main Oct 2, 2023
6 of 7 checks passed
@ShanaLMoore ShanaLMoore deleted the guard-indices-migration branch October 2, 2023 19:51
ShanaLMoore added a commit to scientist-softserv/adventist_knapsack that referenced this pull request Oct 2, 2023
ShanaLMoore added a commit to scientist-softserv/adventist_knapsack that referenced this pull request Oct 2, 2023
ShanaLMoore added a commit to scientist-softserv/adventist_knapsack that referenced this pull request Oct 2, 2023
jeremyf added a commit to scientist-softserv/utk-hyku that referenced this pull request Nov 29, 2023
We're seeing issues with staging indices; namely we're attempting to
generate already exising indices.  By bumping we take advantage of the
resolution of samvera/bulkrax#862

The problem existed in 817e4bb826260c2b402f730e7696dd90027f0c0c but was
resolved in 1fc65e405dee38cba8b363b0837df28f2ef6518d.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
minor-ver for release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants