Skip to content

Conversation

@fschade
Copy link
Contributor

@fschade fschade commented Sep 16, 2025

Description

Recently we've added support for open-search as distributed search backend... the readme was outdated and hadn't mentioned anything about it yet. At the same time, I also reduced the complexity and the level of detail

Related Issue

@fschade fschade force-pushed the docs-service-search-readme branch from 0782c9e to a1a9d67 Compare September 16, 2025 14:28
@fschade fschade requested review from Svanvith and aduffeck and removed request for Svanvith September 16, 2025 14:55
Copy link
Contributor

@aduffeck aduffeck left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just some nitpicks/suggestions. lgtm in general.

fschade and others added 2 commits September 17, 2025 16:04
Co-authored-by: Andre Duffeck <aduffeck@users.noreply.github.com>
@fschade fschade force-pushed the docs-service-search-readme branch from ace7de9 to b0d484b Compare September 17, 2025 14:57
@fschade fschade merged commit c329e7c into opencloud-eu:main Sep 17, 2025
34 of 58 checks passed
@openclouders openclouders mentioned this pull request Sep 17, 2025
1 task
Copy link
Contributor

@micbar micbar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some small questions.

Bleve is a lightweight, embedded full-text search engine written in Go and is the default search backend.
It is easy to set up and requires no additional services to run.

Note that as of now, the search service can not be scaled. Consider using a dedicated hardware for this service in case more resources are needed.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would like to keep the info about the missing scalability

@fschade
Copy link
Contributor Author

fschade commented Sep 17, 2025

Some small questions.

Valid Point; I’ll do a follow up pr Tomorrow, thanks for the Input

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants