-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Support encode for native client #130
Conversation
Unit testing passed. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Well done! 👍
Thanks for implementing this. Regarding the C++ libraries, our preference is folly, boost, and stl, in that order.
So in this case, I would suggest to use folly::dynamic
, unless there is a strong reason that we have to use boost or stl, such as better performance
namespace nebula { | ||
namespace dataman { | ||
|
||
typedef boost::any Value; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would suggest to use folly::dynamic
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for review . Actually I have studied folly::dynamic
but I found it don't have float
data type and can't pick out between double
and float
. So I try to use boost::any
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fair enough :-)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Awesome work! 👍
Unit testing passed. |
Co-authored-by: Shylock Hg <33566796+Shylock-Hg@users.noreply.github.com>
Support encode for native client