MAPREDUCE-1125. SerialUtils.cc: deserializeFloat is out of sync with SerialUtils.hh (Simone Leo via aw)
Project: http://git-wip-us.apache.org/repos/asf/hadoop/repo Commit: http://git-wip-us.apache.org/repos/asf/hadoop/commit/19a0c266 Tree: http://git-wip-us.apache.org/repos/asf/hadoop/tree/19a0c266 Diff: http://git-wip-us.apache.org/repos/asf/hadoop/diff/19a0c266 Branch: refs/heads/HDFS-8707 Commit: 19a0c2660cce0dccf968376bed249de94b58b8a2 Parents: fc7cd46 Author: Allen Wittenauer <a...@apache.org> Authored: Thu Nov 5 16:57:22 2015 -0800 Committer: Allen Wittenauer <a...@apache.org> Committed: Thu Nov 5 16:57:22 2015 -0800 ---------------------------------------------------------------------- hadoop-mapreduce-project/CHANGES.txt | 4 +++- .../src/main/native/utils/api/hadoop/SerialUtils.hh | 1 + .../hadoop-pipes/src/main/native/utils/impl/SerialUtils.cc | 7 +++++++ 3 files changed, 11 insertions(+), 1 deletion(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/hadoop/blob/19a0c266/hadoop-mapreduce-project/CHANGES.txt ---------------------------------------------------------------------- diff --git a/hadoop-mapreduce-project/CHANGES.txt b/hadoop-mapreduce-project/CHANGES.txt index 938509a..17b7f4e 100644 --- a/hadoop-mapreduce-project/CHANGES.txt +++ b/hadoop-mapreduce-project/CHANGES.txt @@ -120,7 +120,6 @@ Trunk (Unreleased) BUG FIXES - MAPREDUCE-6191. Improve clearing stale state of Java serialization testcase. (Sam Liu via Eric Yang) @@ -172,6 +171,9 @@ Trunk (Unreleased) MAPREDUCE-3914. Mismatched free() / delete / delete [] in HadoopPipes (Joe Mudd via aw) + MAPREDUCE-1125. SerialUtils.cc: deserializeFloat is out of sync with + SerialUtils.hh (Simone Leo via aw) + MAPREDUCE-4574. Fix TotalOrderParitioner to work with non-WritableComparable key types. (harsh) http://git-wip-us.apache.org/repos/asf/hadoop/blob/19a0c266/hadoop-tools/hadoop-pipes/src/main/native/utils/api/hadoop/SerialUtils.hh ---------------------------------------------------------------------- diff --git a/hadoop-tools/hadoop-pipes/src/main/native/utils/api/hadoop/SerialUtils.hh b/hadoop-tools/hadoop-pipes/src/main/native/utils/api/hadoop/SerialUtils.hh index cadfd76..ec1838b 100644 --- a/hadoop-tools/hadoop-pipes/src/main/native/utils/api/hadoop/SerialUtils.hh +++ b/hadoop-tools/hadoop-pipes/src/main/native/utils/api/hadoop/SerialUtils.hh @@ -162,6 +162,7 @@ namespace HadoopUtils { void serializeLong(int64_t t, OutStream& stream); int64_t deserializeLong(InStream& stream); void serializeFloat(float t, OutStream& stream); + void deserializeFloat(float& t, InStream& stream); float deserializeFloat(InStream& stream); void serializeString(const std::string& t, OutStream& stream); void deserializeString(std::string& t, InStream& stream); http://git-wip-us.apache.org/repos/asf/hadoop/blob/19a0c266/hadoop-tools/hadoop-pipes/src/main/native/utils/impl/SerialUtils.cc ---------------------------------------------------------------------- diff --git a/hadoop-tools/hadoop-pipes/src/main/native/utils/impl/SerialUtils.cc b/hadoop-tools/hadoop-pipes/src/main/native/utils/impl/SerialUtils.cc index 03d009b..401dfee 100644 --- a/hadoop-tools/hadoop-pipes/src/main/native/utils/impl/SerialUtils.cc +++ b/hadoop-tools/hadoop-pipes/src/main/native/utils/impl/SerialUtils.cc @@ -252,6 +252,13 @@ namespace HadoopUtils { stream.write(buf, sizeof(float)); } + float deserializeFloat(InStream& stream) + { + float f; + deserializeFloat(f, stream); + return f; + } + void deserializeFloat(float& t, InStream& stream) { char buf[sizeof(float)];