dimas-b opened a new pull request #130:
URL: https://github.com/apache/calcite-avatica/pull/130
* Remove unnecessary conversion of Float values to long in
TypedValue.toProto(...). The subsequently invoked writeToProtoWithType(...)
method actually expects float values in this case.
* Make a similar fix for Character values.
* Add Frame serialization round-trip unit tests to cover this failure mode.
* Note: a similar problem appears to still exist for Timestamp and Date/Time
values in this context, but I did not attempt to fix it in this commit
because
there is apparently no use case for putting Timestamp/Date/Time objects
into
Frames. Those values are normally returned as plain numbers in result rows.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
