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

Michael Muller commented on HBASE-13419:
----------------------------------------

Removed the unnecessary println() and also an unnecessary import.

It's possible.  I avoided TestThriftServer (the obvious location) because 
that's a large test that takes a while to run.  The only file there that's not 
a large test is TestCallQueue, which tests functionality unrelated to this 
change.

But I'm happy to put the test anywhere you think best.  Where would you like 
it? 

> Thrift gateway should propagate text from exception causes.
> -----------------------------------------------------------
>
>                 Key: HBASE-13419
>                 URL: https://issues.apache.org/jira/browse/HBASE-13419
>             Project: HBase
>          Issue Type: Improvement
>          Components: Thrift
>            Reporter: Michael Muller
>              Labels: newbie, patch
>             Fix For: 2.0.0, 1.0.1, 1.1.0
>
>         Attachments: ExpandExceptionCauses.patch, ExpandExceptionCauses3.patch
>
>
> Exceptions passed back from the thrift gateway only include the message text 
> of the toplevel exception.  Information from the cause chain is lost.
> In some cases, the top-level exception text is useless but there is some very 
> specific and useful information provided in some of the cause exceptions, so 
> it would be very helpful to flatten all of the messages into the exception 
> text returned to the user.



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

Reply via email to