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

feat(search): search with meilisearch. #6060

Merged
merged 5 commits into from
Feb 23, 2024
Merged

feat(search): search with meilisearch. #6060

merged 5 commits into from
Feb 23, 2024

Conversation

r27153733
Copy link
Contributor

#6059 #1740
支持meilisearch

Copy link

welcome bot commented Feb 19, 2024

Thanks for opening this pull request! Please check out our contributing guidelines.

@r27153733
Copy link
Contributor Author

r27153733 commented Feb 19, 2024

meilisearch/meilisearch#3751
meilisearch 因为性能原因暂不支持STARTSWITH。所以没有很好的办法实现Del接口。
目前有两种方式可以曲线救国。

  1. 用 dfs + get 获取所有可能的 parent 最后一起删除。(内存问题,parent数量可能非常多。)
  2. dfs的过程删除。(不能保证原子性,删一半出错了怎么办?)
  3. 不支持这个功能

@r27153733
Copy link
Contributor Author

r27153733 commented Feb 22, 2024

@xhofe 考虑合并一下?

@xhofe xhofe merged commit f1979a8 into alist-org:main Feb 23, 2024
2 of 3 checks passed
@anwen-anyi
Copy link
Member

Meilisearch 如何启用? 直接启用会提示错误,还是说需要其它应用搭建然后搭配使用
image

@xhofe
Copy link
Member

xhofe commented Feb 24, 2024

EtherealAO pushed a commit to EtherealAO/alist that referenced this pull request Apr 12, 2024
* feat(search): search with meilisearch.

* feat(search): meilisearch supports auto update.

* chores: remove utils.Log.

* fix(search): the null pointer caused by deleting non-existing file/folder indexes.

---------

Co-authored-by: Andy Hsu <i@nn.ci>
EtherealAO pushed a commit to EtherealAO/alist that referenced this pull request Apr 12, 2024
* feat(search): search with meilisearch.

* feat(search): meilisearch supports auto update.

* chores: remove utils.Log.

* fix(search): the null pointer caused by deleting non-existing file/folder indexes.

---------

Co-authored-by: Andy Hsu <i@nn.ci>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants