-
Notifications
You must be signed in to change notification settings - Fork 349
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
Making Trie Serializable #28
Comments
+1 |
1 similar comment
+1 |
I've just sent a new pull request for that. It's based on the work done in this older pull request. |
@humanzz, any chance of getting a fix on the pull request so it passes the check-in requirements? |
any reason holding back on incorporating the pull request? |
@DNWEIJ the PR fails to pass all the checks because of merge conflicts. Once those conflicts are resolved, I'll merge the PR. |
@DNWEIJ -- do you know anyone who'd be interested in finishing this PR? |
For instance in the case of caching to some third party provider like memcached (and since building a
Trie
is not a cheap computation),Trie
should implementSerializable
interface. Could you provide this feature?The text was updated successfully, but these errors were encountered: