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

[C++] Implement meta string encoding algorithm for C++ #1543

Open
chaokunyang opened this issue Apr 19, 2024 · 4 comments
Open

[C++] Implement meta string encoding algorithm for C++ #1543

chaokunyang opened this issue Apr 19, 2024 · 4 comments
Assignees
Labels
enhancement New feature or request good first issue Good for newcomers

Comments

@chaokunyang
Copy link
Collaborator

Is your feature request related to a problem? Please describe.

We've implemented meta string encoding algorithm in https://fury.apache.org/docs/specification/fury_xlang_serialization_spec#meta-string for java in #1514 , it's time to implement it in c++.

Describe the solution you'd like

Java implementation in #1514 can be taken as a reference. But note that the meta string encoding algorithm is used for encode field name only, so the special charater can't be . or $, thus the implementation will be simpler

Additional context

#1413

@chaokunyang chaokunyang added enhancement New feature or request good first issue Good for newcomers labels Apr 19, 2024
@heliang666s
Copy link

please assign me,thanks!

@chaokunyang
Copy link
Collaborator Author

please assign me,thanks!

Great! Feel free to ask if you have further questions when implement this feature.

@Yangsx-1
Copy link

Hi, i'm new in here and want to do some work. It seems to be inactivated for a while and i want to try this. Could you assign this task to me? @chaokunyang

@chaokunyang chaokunyang assigned Yangsx-1 and unassigned heliang666s May 23, 2024
@chaokunyang
Copy link
Collaborator Author

Hi, i'm new in here and want to do some work. It seems to be inactivated for a while and i want to try this. Could you assign this task to me? @chaokunyang

Assigned, thanks for contributing to Apache Fury @Yangsx-1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

3 participants