From 450bc69248043f0ea5ca336d786583ebf3e9d826 Mon Sep 17 00:00:00 2001 From: Jens Geyer Date: Tue, 3 Dec 2019 23:28:03 +0100 Subject: THRIFT-4915 Deserializing double into OrderedFloat always returns zero when using TCompactProtocol Documentation fix Patch: Jens Geyer --- doc/specs/thrift-compact-protocol.md | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) (limited to 'doc') diff --git a/doc/specs/thrift-compact-protocol.md b/doc/specs/thrift-compact-protocol.md index 02467dd19..6be2a62f8 100644 --- a/doc/specs/thrift-compact-protocol.md +++ b/doc/specs/thrift-compact-protocol.md @@ -97,8 +97,9 @@ Where: ### Double encoding Values of type `double` are first converted to an int64 according to the IEEE 754 floating-point "double format" bit -layout. Most run-times provide a library to make this conversion. Both the binary protocol as the compact protocol then -encode the int64 in 8 bytes in big endian order. +layout. Most run-times provide a library to make this conversion. But while the binary protocol encodes the int64 +in 8 bytes in big endian order, the compact protocol encodes it in little endian order - this is due to an early +implementation bug that finally became the de-facto standard. ### Boolean encoding -- cgit v1.2.1