Closed Yiwen-Gao closed 1 year ago
@Yiwen-Gao is attempting to deploy a commit to the Greg's Team Team on Vercel.
A member of the Team first needs to authorize it.
This looks great, thanks for contributing! I'll test it out in the next day along with @TimDaub's indexer PR!
@Yiwen-Gao amazing, thanks for completing it :)
The latest updates on your projects. Learn more about Vercel for Git ↗︎
Name | Status | Preview | Comments | Updated (UTC) |
---|---|---|---|---|
searchcaster | ✅ Ready (Inspect) | Visit Preview | 💬 Add feedback | Jul 16, 2023 10:54pm |
So sorry for the delay here - I finally tried everything out locally and the improvement is super noticeable. I made a slight modification to the textSearch
config to make it behave similar to before (as a keyword search vs phrases/full sentences - the results were quite different), but looks great!! Thanks so much for doing this!
After https://github.com/gskril/farcaster-indexer/pull/20 lands and the database is updated, we should update the queries to take advantage of full text search.
I think this would address issues raised in https://github.com/gskril/searchcaster/issues/19. But I don't think FTS supports backticks, so I wasn't able to reproduce https://github.com/gskril/searchcaster/issues/22.
https://user-images.githubusercontent.com/32348009/233812488-5dc98dfd-d629-42b3-9b91-533193fdda7b.mov