Yifan Zhang has uploaded this change for review. ( http://gerrit.cloudera.org:8080/14114
Change subject: Add ExtractFloat method to JsonReader and improve ExtractDouble method ...................................................................... Add ExtractFloat method to JsonReader and improve ExtractDouble method ExtractDouble method should not only parse double values, int values that can be losslessly converted to double should also be parsed. Though there are already IsLosslessDouble()/IsLosslessFloat() methods in rapidjson library, the implementation seems have problems. This patch defined new methods to determine whether a value can be losslessly converted to a double/float value. Change-Id: Ica0a04fb21daceaa44233995a42a60686f59361b --- M src/kudu/tools/tool_action_table.cc M src/kudu/util/jsonreader-test.cc M src/kudu/util/jsonreader.cc M src/kudu/util/jsonreader.h 4 files changed, 108 insertions(+), 6 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/14/14114/1 -- To view, visit http://gerrit.cloudera.org:8080/14114 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: Ica0a04fb21daceaa44233995a42a60686f59361b Gerrit-Change-Number: 14114 Gerrit-PatchSet: 1 Gerrit-Owner: Yifan Zhang <[email protected]>
