-
Notifications
You must be signed in to change notification settings - Fork 0
Solr fulltext indexing #45
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
base: main
Are you sure you want to change the base?
Conversation
This uses the collective.solr provided solr version and config, reformatted by pre-commit in recensio.plone.
This backports the solr schema configuration from the old recensio.deployment production generated solr schema.xml. To be cleaned up.
profiles/development.cfg
Outdated
|
|
||
| [sources] | ||
| recensio.plone = git https://github.com/syslabcom/recensio.plone.git pushurl[email protected]:syslabcom/recensio.plone.git branch=main | ||
| recensio.plone = git https://github.com/syslabcom/recensio.plone.git pushurl[email protected]:syslabcom/recensio.plone.git branch=solr |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This PR should be merged without this change after syslabcom/recensio.plone#204 is merged
|
Ready for merge as soon as syslabcom/recensio.plone#204 is merged and the checkout branch for that is changed to |
| /parts/ | ||
| /secret.cfg | ||
| /solr/ | ||
| /solr-9.9.0.tgz |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| /solr-9.9.0.tgz |
Refs https://github.com/syslabcom/scrum/issues/3147
A full migration run indexing all production content takes about 50 minutes on my machine. A handful of documents throw errors, but that is to be expected.
Note that following this discussion, this PR does not use
collective.recipe.solrinstancewhich tries to manage the solr configuration via buildout; instead this PR uses kitconcept.recipe.solr which basically downloads the release, creates a core, and copies in the git managed config files. Note that the fallback there which tries to copy the config from the recipe egg is broken, but nevermind we're providing our own config in this PR.I'm building solr only in development. Anybody is free to deploy by either downloading Solr and copying in the config themselves, or by running a docker and bind-mounting the config directory, or by using buildout.
Note that the Solr buildout and config here are identical to the one used in CI in
recensio.plone, except for the Tika fix in supervisord which is not present there. For more background about the Tika fulltext indexing fix, see the collective.solr issue I opened.This PR is for review and interactive testing, not for merge yet, since it hardcodes the
solrbranch forrecensio.plone.