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

Index embargo_length as a singluar, searchable, stored value in Solr #2095

Merged
merged 1 commit into from
Feb 25, 2021

Conversation

mark-dce
Copy link
Contributor

Previously embargo_length was indexed as a stored, but not indexed,
symbol (_ssm). This prevents searching Solr for the attribute,
which has made debugging certain embargo and graduation bugs difficult.

The changes index the attribute as a single-value symbol (_ssi).
Since the student can only choose one value for the requested embargo
duration, this aligns the data model more closely with the problem domain.

Previously `embargo_length` was indexed as a stored, but not indexed,
symbol (`_ssm`).  This prevents searching Solr for the attribute,
which has made debugging certain embargo and graduation bugs difficult.

The changes index the attribute as a single-value symbol (`_ssi`).
Since the student can only choose one value for the requested embargo
duration, this aligns the data model more closely with the problem domain.
@coveralls
Copy link

coveralls commented Feb 25, 2021

Coverage Status

Coverage remained the same at 87.607% when pulling 8952d1b on index-singular-embargo-length into 0f6b7f9 on master.

@maxkadel maxkadel merged commit 1ac3eca into master Feb 25, 2021
@maxkadel maxkadel deleted the index-singular-embargo-length branch February 25, 2021 14:55
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

Successfully merging this pull request may close these issues.

None yet

3 participants