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

[docs-only] search service readme #5503

Merged
merged 1 commit into from
Feb 7, 2023
Merged

[docs-only] search service readme #5503

merged 1 commit into from
Feb 7, 2023

Conversation

fschade
Copy link
Contributor

@fschade fschade commented Feb 2, 2023

I wrote down what comes to mind about the search service, @mmattel please read what you think about it.
I haven't proofread much, please check for spelling mistakes too ;)

@mmattel
Copy link
Contributor

mmattel commented Feb 6, 2023

@aduffeck @fschade can you have a look on the text if it is correct from a technical pov.
@EParzefall will do a language review.

services/search/README.md Outdated Show resolved Hide resolved
services/search/README.md Outdated Show resolved Hide resolved
services/search/README.md Outdated Show resolved Hide resolved
services/search/README.md Outdated Show resolved Hide resolved
services/search/README.md Outdated Show resolved Hide resolved
services/search/README.md Outdated Show resolved Hide resolved
services/search/README.md Outdated Show resolved Hide resolved
services/search/README.md Outdated Show resolved Hide resolved
services/search/README.md Outdated Show resolved Hide resolved
services/search/README.md Outdated Show resolved Hide resolved
@mmattel mmattel marked this pull request as ready for review February 6, 2023 15:36
@Jip-Hop
Copy link

Jip-Hop commented Feb 6, 2023

Can the search service be disabled (temporarily)? @butonic mentions he has no search service running. Perhaps disabling search indexing would speed up initial upload.

@mmattel
Copy link
Contributor

mmattel commented Feb 6, 2023

Can the search service be disabled (temporarily)? @butonic mentions he has no search service running. Perhaps disabling search indexing would speed up initial upload.

  • The search service consumes events and does not block other tasks (no wait for finished indexing).
  • When the search service runs on the same hardware, it is competing on ressources as all others do.
  • Depending on the deployment method, services can be started and stopped individually. See the admin docs at General Info for more details. But it would not make sense to temporarily stop search as it gets info via the event bus when to trigger indexing, see the text in this repo for details.

@mmattel
Copy link
Contributor

mmattel commented Feb 7, 2023

Note: with commit add resource deleted (many thanks for the info @aduffeck), item number 4 of the referenced issue #5515 is fixed. I will add a note there too.

@mmattel
Copy link
Contributor

mmattel commented Feb 7, 2023

I will do a squash to ease backporting to 2.0.0 @kobergj

@mmattel mmattel merged commit 9347cc2 into master Feb 7, 2023
@delete-merged-branch delete-merged-branch bot deleted the search-readme branch February 7, 2023 08:56
ownclouders pushed a commit that referenced this pull request Feb 7, 2023
Merge: d27dfa4 8dda0c6
Author: Martin <github@diemattels.at>
Date:   Tue Feb 7 09:56:37 2023 +0100

    Merge pull request #5503 from owncloud/search-readme

    [docs-only] search service readme
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants