[ 
https://issues.apache.org/jira/browse/CALCITE-1050?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15964995#comment-15964995
 ] 

Julian Hyde edited comment on CALCITE-1050 at 4/11/17 9:38 PM:
---------------------------------------------------------------

Thanks so much for this, [~elserj]!

Review comments:
* In {{jdbcToSerial}} it doesn't look as if it handles null {{Timestamp}} 
values.
* In {{AvaticaUtils}}, the {{getNonPrimitiveRep}} and {{getSerializedRep}} 
methods can be moved to {{Rep}}, and maybe rationalized a bit.


was (Author: julianhyde):
Thanks so much for this, @elserj!

Review comments:
* In {{jdbcToSerial}} it doesn't look as if it handles null {{Timestamp}} 
values.
* In {{AvaticaUtils}}, the {{getNonPrimitiveRep}} and {{getSerializedRep}} 
methods can be moved to {{Rep}}, and maybe rationalized a bit.

> Avatica can't serialize java.sql.Array
> --------------------------------------
>
>                 Key: CALCITE-1050
>                 URL: https://issues.apache.org/jira/browse/CALCITE-1050
>             Project: Calcite
>          Issue Type: Improvement
>          Components: avatica
>    Affects Versions: 1.5.0
>            Reporter: Lukas Lalinsky
>            Assignee: Josh Elser
>             Fix For: avatica-1.10.0
>
>
> As far as I can see, there is no way to serialize arrays in the Avatica RPC.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

Reply via email to