vaticle / typedb-driver

TypeDB Drivers for Rust, Python, Java, Node.js, C, C++, and C#.
https://typedb.com
Apache License 2.0
30 stars 32 forks source link

Correctly handle fetch JSON numbers in Java and C++ drivers #603

Closed dmitrii-ubskii closed 4 months ago

dmitrii-ubskii commented 4 months ago

Usage and product changes

When receiving the fetch response across the FFI boundary as JSON, Java and C++ drivers would incorrectly parse numbers: Java as a single precision float, and C++ as an integer. This PR fixes both those issues.

Implementation

Fixes #601

vaticle-bot commented 4 months ago

PR Review Checklist

Do not edit the content of this comment. The PR reviewer should simply update this comment by ticking each review item below, as they get completed.


Trivial Change

Code

Architecture