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

liang xie commented on HBASE-6882:
----------------------------------

Hi Mikhail, seems attached file is not for current community TRUNK version? 
since i saw :
{code:title=Hbase.thrift|borderStyle=solid}
 exception IOError {
   1: string message,
-  2: i64 backoffTimeMillis
+  2: i64 backoffTimeMillis,
+  3: string exceptionClass
 }
{code} 

there is no backoffTimeMillis parameter in struct IOError on current trunk code

and another thing, do we encourage using thrift2 more than thrift right now ? 
if that's right, maybe changing thrift2's TIOError is great ? 
                
> Thrift IOError should include exception class
> ---------------------------------------------
>
>                 Key: HBASE-6882
>                 URL: https://issues.apache.org/jira/browse/HBASE-6882
>             Project: HBase
>          Issue Type: Improvement
>            Reporter: Mikhail Bautin
>            Assignee: Mikhail Bautin
>         Attachments: D5679.1.patch
>
>
> Return exception class as part of IOError thrown from the Thrift proxy or the 
> embedded Thrift server in the regionserver.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to