-
Notifications
You must be signed in to change notification settings - Fork 2.9k
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
Change EmojiTrie implementation to SuffixTree #51148
Comments
cc @roryabraham |
Hey @hannojg, I have been very passionate Competitive Programmer back during my college days. |
I think this is more of an internal ticket. The suffix tree implementation has already been built by our internal team and this is a follow up issue from one of our PRs. |
Alright, |
For this issue ticket we have implemented a very efficient/fast suffix tree implementation:
PR:
It was discussed that we want to use the implementation of the SuffixTree for the emoji search as well:
This enables:
The text was updated successfully, but these errors were encountered: