[C++] Add the basic row format serializer for C++ class types via reflection#1144
[C++] Add the basic row format serializer for C++ class types via reflection#1144chaokunyang merged 7 commits intoapache:mainfrom
Conversation
|
Great work! @PragmaTwice . We use encoder for row format encoding in Java and Python, and use serialization for object graph serialization. Do we need to keep consistent with Java and Python naming? |
Sure! Let me rename it to |
|
Hi @chaokunyang , I think I've finished the implementation phase of this patch now. Check row_encoder_test.cc for a simple use case. Current only basic types are supported, and just a raw trait API is provided, I will gradually support other features according to #1145. Feel free to review it when you are free : ) |
|
@PragmaTwice, very impressive and professional work, thank you so much for submitting the CPP serialization implementation for Fury. This is a very big step for fury cpp support, it not only enhances the project but also helps the development of our community. Can't tell how excited I am. |
|
Can we add a simple user doc to |
chaokunyang
left a comment
There was a problem hiding this comment.
LGTM overall, I left some minor comments
I think the current interface (i.e. And maybe we can add these documentation after that. |
Got it |
What do these changes do?
In this PR, we will include a simple row format serializer for class types.
Currently, it only supports basic field types (support for nested types will be added in future PRs).
Since the object graph protocol is not implemented in C++, we only support row format.
However, I believe the serialization framework can be easily extended to other format.
Related issue number
#1145
Check code requirements