Skip to content

UBER-921: Improve full text search#3848

Merged
haiodo merged 1 commit into
mainfrom
UBER-921
Oct 17, 2023
Merged

UBER-921: Improve full text search#3848
haiodo merged 1 commit into
mainfrom
UBER-921

Conversation

@haiodo

@haiodo haiodo commented Oct 17, 2023

Copy link
Copy Markdown
Contributor

Contribution checklist

Brief description

Not fully ideal, since elastic not filter, or I do not understand how to filter, and only move documents with matched fields more in rank.

Checklist

  • - UI test added to added/changed functionality?
  • - Screenshot is added to PR if applicable ?
  • - Does the code work? Check whether function and logic are correct.
  • - Does Changelog.md is updated with changes?
  • - Does the translations are up to date?
  • - Does it well tested?
  • - Tested for Chrome.
  • - Tested for Safari.
  • - Go through the changed code looking for typos, TODOs, commented LOCs, debugging pieces of code, etc.
  • - Rebase your branch onto master and upstream branch
  • - Is there any redundant or duplicate code?
  • - Are required links are linked to PR?
  • - Does new code is well documented ?

Related issues

A list of closed updated issues

Signed-off-by: Andrey Sobolev <haiodo@gmail.com>
@haiodo
haiodo requested a review from BykhovDenis October 17, 2023 06:26
Comment thread server/core/src/fulltext.ts
@haiodo
haiodo merged commit c199ab8 into main Oct 17, 2023
@haiodo
haiodo deleted the UBER-921 branch October 17, 2023 08:22
nestoragent pushed a commit that referenced this pull request Oct 17, 2023
Signed-off-by: Andrey Sobolev <haiodo@gmail.com>
Signed-off-by: Alex Velichko <nestor_007@mail.ru>
nestoragent added a commit that referenced this pull request Oct 17, 2023
nestoragent added a commit that referenced this pull request Oct 17, 2023
nestoragent added a commit that referenced this pull request Oct 17, 2023
MichaelUray pushed a commit to MichaelUray/huly-platform that referenced this pull request Jul 9, 2026
Signed-off-by: Andrey Sobolev <haiodo@gmail.com>
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.

2 participants