Carel created THRIFT-5772:
-----------------------------

             Summary: Add UUID support for C++
                 Key: THRIFT-5772
                 URL: https://issues.apache.org/jira/browse/THRIFT-5772
             Project: Thrift
          Issue Type: New Feature
            Reporter: Carel


Starting the discussion to add UUID support for C++.

I have started with an implementation (See PR linked soon)

Open points for discussion: 
# Is there perhaps a need to add a wrapper class for UUID?
## I started off with {{std::string}} for ease-of-use
## 'ease-of-use' has the drawback of ease of miss-use... 
## Something lightweight as a strong wrapper to wrap a `{{uint8_t data[16];}}`
### A wrapper gives us the ability to add utility functions, like `to_string()` 
and `from_string()`
### Can use {{boost::uuids::uuid}} in the implementation, hidden from users 
(see current implementation)
## I will see if I can't implement something while this PR is looked at.
# Can someone help with some cross tests
## With the absence of working dockers I am really struggling to get other 
languages compiled.
## I don't have enough experience with other languages to effectively do this
## I will probably be able to add Java tests if there is something to build on 
(pointers needed)
# How important is the JSON side? 
## I did add the support but I see only go has some support for it. 




--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to