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

Ted Malaska commented on KUDU-1267:
-----------------------------------

Build passed http://104.196.14.100/job/kudu-gerrit/1412/

But I didn't add a check for Long because getLong can be called for String, 
Long, and Timestamp

How would you like me to handle a check for long?  Or should I just not add 
that check because it is out of scope for this Jira

Also I would like to make addition jira after this.  To make the executions 
thrown by checks return the column name and not just the column indexes.



> Calling getString() on RowResult for wrong type gave weird exception
> --------------------------------------------------------------------
>
>                 Key: KUDU-1267
>                 URL: https://issues.apache.org/jira/browse/KUDU-1267
>             Project: Kudu
>          Issue Type: Bug
>          Components: client
>    Affects Versions: 0.5.0
>            Reporter: Todd Lipcon
>            Assignee: Ted Malaska
>            Priority: Critical
>              Labels: newbie
>         Attachments: KUDU-1267.1.patch
>
>
> When calling getString(idx) on a column that isn't a string, it yields a 
> bizarre exception.
> Similarly, calling getInt() on a 'short' column gives silently wrong results.
> For all cases we should have a check that the type matches the data to 
> prevent confused users getting wrong results.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to