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

Feature: Support CompactString #30

Open
PlainBane opened this issue Jul 28, 2024 · 2 comments
Open

Feature: Support CompactString #30

PlainBane opened this issue Jul 28, 2024 · 2 comments

Comments

@PlainBane
Copy link

Hi,
Could you consider adding support for CompactString as an alternative for String in structs with small string values.
It would add a bit of performance improvement when using structs.

@ohsayan
Copy link
Member

ohsayan commented Dec 14, 2024

Can you please explain how you plan to use it? I'll see if I can get it implemented.

@PlainBane
Copy link
Author

I use it in Axum http api structs (To optimize small string values). So if I Serialize / Deserialize struct, there are lots of CompactString instead of String. If I then use skytable to cache some of this data, I end up having to convert CompactString -> String and back. I could try to add it myself and do pr.
Thanks.

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

No branches or pull requests

2 participants