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.

Sign up for GitHub

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

Jump to bottom

Valkyrie::Persistence::Solr::ModelConverter::StringPropertyValue#fields value length cutoff should be configurable (and logged?) #961

Closed
dchandekstark opened this issue Sep 4, 2024 · 0 comments · Fixed by #964
Closed

Valkyrie::Persistence::Solr::ModelConverter::StringPropertyValue#fields value length cutoff should be configurable (and logged?) #961

dchandekstark opened this issue Sep 4, 2024 · 0 comments · Fixed by #964

Comments

@dchandekstark
Copy link
Member

dchandekstark commented Sep 4, 2024 •
edited
Loading

Values >1000 chars in length are only indexed using the _tsim field. I don't know exactly what the motivation was initially, but I don't see the reason to limit the length of _tesim, and it does have detrimental effects.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Adds config setting index_tsim_only_threshold dchandekstark/valkyrie
1 participant
@dchandekstark