-
-
Notifications
You must be signed in to change notification settings - Fork 2k
docs: Add tutorial for configuring Dovecot FTS with Solr #4070
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
Conversation
Describe how to use solr as a dovecot fts backend.
Co-authored-by: Casper <[email protected]>
|
georglauterbach
left a comment
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.
After @casperklein's and my feedback has been addressed, you will still need to adjust the mkdocs.yml to integrate your new page into our documentation.
The PR itself is still in a rough shape; I will also request @polarathene for a final review, as he has the most experience with our docs. I don't want him to have a third look currently, though; having the last word and final review is appropriate IMO.
Co-authored-by: Casper <[email protected]>
Co-authored-by: Casper <[email protected]>
Co-authored-by: Casper <[email protected]>
Co-authored-by: Casper <[email protected]>
Co-authored-by: Casper <[email protected]>
Co-authored-by: Casper <[email protected]>
Co-authored-by: Casper <[email protected]>
Co-authored-by: Georg Lauterbach <[email protected]>
Co-authored-by: Georg Lauterbach <[email protected]>
syntax highlighting
|
@polarathene go ahead and have a look :) I suggest you immediately apply feedback that you have 🚀 @beertje44 you can now already take a look at the preview we generate. Look out for comments from the github-actions bot, like this one. |
|
Current preview of new docs page: https://pullrequest-4070--dms-doc-previews.netlify.app/examples/tutorials/dovecot-solr/ I have revised the PR a bit:
There is no mention for the I'll add a new commit to advise on including the package and ARM64 caveat, then this should be good for merging. |
polarathene
left a comment
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.
I'll apply these.
|
Documentation preview for this PR is ready! 🎉 Built with commit: 4260233 |
|
Thanks everyone! This guide seems to be missing some advice that was detailed by @erm967 in #4052 (comment) , is that not required for some reason? I suspect that is one of the maintenance caveats to keep in mind as the external resource won't be tied to a specific DMS release (at least if this new doc page is not actively maintained across releases). I've left the referenced feature request open, but I've also included a link on the new docs page for any interested users to express their interest in the package. I think we've covered why we no longer include the package and the workaround for Solr users is quite simple as covered in the docs. Closing that feature request with this PR is acceptable. It can always be opened again in future if relevant. |
Simply including the right jars in a lib dir inside the solr core dir you have to create anyway is sufficient to make the dovecot schema work as is. Imho this is a better solution for most users then having to apply a patch that will change for every solr minor release. I've included a |
Description
Tutorial on using standard apache solr container with dovecot provided solr schema as dovecot fts backend in dms.
ref: #4025
Type of change
Checklist
docs/)CHANGELOG.md