[ 
https://issues.apache.org/jira/browse/CASSANDRA-202?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12715139#action_12715139
 ] 

Jonathan Ellis commented on CASSANDRA-202:
------------------------------------------

> The problem with catching only at CFS level is that one can't get parameters 
> other than key and cf. Other parameters are useful for debugging purpose. 

The other parameters aren't going to be involved in an IOException so the 
argument for 5x code duplication for the sake of including them  is weak.

> Refactor different getRow calls in Table.java to more specific name and add 
> more logging information
> ----------------------------------------------------------------------------------------------------
>
>                 Key: CASSANDRA-202
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-202
>             Project: Cassandra
>          Issue Type: Improvement
>            Reporter: Jun Rao
>            Assignee: Jun Rao
>         Attachments: issue202.patchv1, issue202.patchv2
>
>
> The Table class has five different getRow methods. We need to change them to 
> more appropriate names. Also, when an IOException occurs, it would be useful 
> to log the parameters of the request. 

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to