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

Meilisearch - Added metadata as filterable attributes and polling on enqueued tasks #3042

Conversation

MohamedYasserOaf
Copy link
Contributor

@MohamedYasserOaf MohamedYasserOaf commented Aug 20, 2024

Hello there! back again with another improvement for Meilisearch component, I added metadata of the input documents to the metadata of meilisearch hits so no metadata information is lost, I also added all of the metadata content as filterable attributes to enable meilisearch filtering through the component. Moreover,some of meilisearch requests return an "enqueuedTask" , it creates an enqueuedTask depending on the request (adddocuments/update/delete/create-index etc...), so it was necessary to add proper polling on these created tasks to ensure correct upserting of large datasets and to avoid taking any action before these tasks are fully completed.

Example: loop on add documents enqueuedTask to ensure that the upsert function does not end before adding documents already happened on meilisearch cloud

This is the shape of the updated component, it is fully tested and working on production environment, feel free to ask about any of the proposed changes. Thanks

meilisearch1-1

@MohamedYasserOaf
Copy link
Contributor Author

I also added a crucial API request for meilisearch, it enables vector search feature on a given host

Copy link
Contributor

@HenryHengZJ HenryHengZJ left a comment

Choose a reason for hiding this comment

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

thank you @MohamedYasserOaf !

@HenryHengZJ HenryHengZJ merged commit 0f58d31 into FlowiseAI:main Aug 23, 2024
2 checks passed
@MohamedYasserOaf
Copy link
Contributor Author

you are welcome @HenryHengZJ ! thanks for your review 👍

patrickreinan pushed a commit to patrickreinan/Flowise that referenced this pull request Sep 3, 2024
…enqueued tasks (FlowiseAI#3042)

* added polling for enqueued tasks,metadata filterable

* delete index feature with polling on task

* added search filter interface

* Update pnpm-lock.yaml

---------

Co-authored-by: Henry Heng <henryheng@flowiseai.com>
patrickreinan pushed a commit to patrickreinan/Flowise that referenced this pull request Sep 3, 2024
…enqueued tasks (FlowiseAI#3042)

* added polling for enqueued tasks,metadata filterable

* delete index feature with polling on task

* added search filter interface

* Update pnpm-lock.yaml

---------

Co-authored-by: Henry Heng <henryheng@flowiseai.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