proost commented on issue #533: URL: https://github.com/apache/datasketches-cpp/issues/533#issuecomment-5940396879
I have short time about two weeks. What i think: 1. We should change policy that using utf-8 as default in the AoS sketch. You can find last discussion. We need to discuss policy again. 2. In c++, without third party dependencies, it is difficult to decode as utf-16LE(standard header is deprecated in c++17, removed in c++26). So as you reviewed in the previous merged PR, we need to add new dependency. It needs discussion too. Because of this two reasons, i suggest revert my AoS sketch PR. Releasing 5.3.0 is more important than introducing AoS sketch. I will rewrite completely binary compatible AoS sketch. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
