[THRIFT-5772] Add UUID support for C++

• Type: New Feature
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: 0.21.0
• Component/s: C++ - Compiler
• Assignee: Carel
• Reporter: Carel
• Created: 27/Mar/24 19:11
• Updated: 21/Sep/24 13:30
• Resolved: 07/Apr/24 20:44

Description

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

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

Open points for discussion:

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

Issue Links

• is a child of: THRIFT-5587 Introduce uuid as additional builtin type
• links to: GitHub Pull Request #2952