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

Added SOLR support #1297

Closed
wants to merge 1 commit into from
Closed

Added SOLR support #1297

wants to merge 1 commit into from

Conversation

ajgon
Copy link
Contributor

@ajgon ajgon commented Dec 26, 2019

What type of PR?

Feature

What does this PR do?

Adds support for full text search via SOLR to dovecot (imap).

Prerequistes

Before we can consider review and merge, please make sure the following list is done and checked.
If an entry in not applicable, you can check it or remove it from the list.

  • In case of feature or enhancement: documentation updated accordingly
  • Unless it's docs or a minor change: add changelog entry file.

PS. changelog guide is 404 😏

@mergify
Copy link
Contributor

mergify bot commented Dec 26, 2019

Thanks for submitting this pull request.
Bors-ng will now build test images. When it succeeds, we will continue to review and test your PR.

bors try

Note: if this build fails, read this.

bors bot added a commit that referenced this pull request Dec 26, 2019
@bors
Copy link
Contributor

bors bot commented Dec 26, 2019

try

Build failed

@ajgon
Copy link
Contributor Author

ajgon commented Dec 26, 2019

bors retry

@bors
Copy link
Contributor

bors bot commented Dec 26, 2019

🔒 Permission denied

Existing reviewers: click here to make ajgon a reviewer

@der-eismann
Copy link
Contributor

Can you also run this with solr 8?

@kaiyou
Copy link
Member

kaiyou commented Jan 13, 2020

We decided in #1176 that we were going to go with ES instead, and not have too many options to maintain until one of them is mastered.

bors bot added a commit that referenced this pull request Feb 1, 2020
1320: Add xapian full-text-search plugin to dovecot r=mergify[bot] a=Nebukadneza

## What type of PR?
Enhancement

## What does this PR do?
Currently we are not able to offer our users a FTS experience after the
demise of lucene due to unfixed coredumps with musl/alpine.
We now add lucene, the only remaining maintained small/lean FTS plugin
for dovecot. It is quite simple to add to our stack: A two-stage docker
build is used to compile the fts plugin in the first stage, and copy
over only the resulting plugin-artifact to the second stage, which is
our usual dovecot container. Configuration is also minimal.

There was a upstream issue where bodies were not able to be searched for subwords, but fortunately it was fixed quite quickly. We currently need to wait for a new release to use a stable tag in our `Dockerfile`.

### Related issue(s)
- #1176
- #1297
- #751
- **Upstream-issues which is the cause for the `TODO` in the `Dockerfile`**: grosjo/fts-xapian#32

## Prerequistes
- [ ] Wait for upstream to prepare new release after grosjo/fts-xapian#32 — so that we can use a stable tag in our `Dockerfile`
- [ ] In case of feature or enhancement: documentation updated accordingly
- [ ] Unless it's docs or a minor change: add [changelog](https://mailu.io/master/contributors/guide.html#changelog) entry file.


Co-authored-by: Dario Ernst <dario@kanojo.de>
bors bot added a commit that referenced this pull request Mar 10, 2020
1320: Add xapian full-text-search plugin to dovecot r=mergify[bot] a=Nebukadneza

## What type of PR?
Enhancement

## What does this PR do?
Currently we are not able to offer our users a FTS experience after the
demise of lucene due to unfixed coredumps with musl/alpine.
We now add lucene, the only remaining maintained small/lean FTS plugin
for dovecot. It is quite simple to add to our stack: A two-stage docker
build is used to compile the fts plugin in the first stage, and copy
over only the resulting plugin-artifact to the second stage, which is
our usual dovecot container. Configuration is also minimal.

There was a upstream issue where bodies were not able to be searched for subwords, but fortunately it was fixed quite quickly. We currently need to wait for a new release to use a stable tag in our `Dockerfile`.

### Related issue(s)
- #1176
- #1297
- #751
- **Upstream-issues which is the cause for the `TODO` in the `Dockerfile`**: grosjo/fts-xapian#32

## Prerequistes
- [ ] Wait for upstream to prepare new release after grosjo/fts-xapian#32 — so that we can use a stable tag in our `Dockerfile`
- [ ] In case of feature or enhancement: documentation updated accordingly
- [ ] Unless it's docs or a minor change: add [changelog](https://mailu.io/master/contributors/guide.html#changelog) entry file.


Co-authored-by: Dario Ernst <dario@kanojo.de>
Co-authored-by: Dario Ernst <dario.ernst@rommelag.com>
@muhlemmer
Copy link
Member

@kaiyou should this be closed?

@muhlemmer muhlemmer added the status/blocked This will block mergify until the label is removed. label Apr 30, 2020
@Nebukadneza
Copy link
Member

As we took in xapian recently, and this is quite stale / would need a batch of rebasing, i’m with @muhlemmer for closing.

@ajgon — sorry for that, and thank you a lot for your contribution, which served well as a discussion-base to bring any FTS into mailu again!

@Nebukadneza Nebukadneza closed this Aug 8, 2020
@ajgon
Copy link
Contributor Author

ajgon commented Aug 10, 2020

@Nebukadneza No worries, actually it's for the best. I used this setup successfully, for the past 8 months, but eventually have to kill it recently - because JAVA was unsatisfied daemon eating my CPU cycles like crazy (and keeping rig temperatures high). So double thumbs up for xapian, can't wait to see it in action!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status/blocked This will block mergify until the label is removed.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants