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

Chia-Ping Tsai commented on HBASE-20055:
----------------------------------------

{quote}What made the tests pass [~uagashe] ?
{quote}
The error message is changed by the patch so the assert check fails.
{code:java}
} catch (ExecutionException ie) {
System.out.println(ie);
assertTrue("Exception should mention regions not online",
- StringUtils.stringifyException(ie.getCause()).contains("regions not online")
+ StringUtils.stringifyException(ie.getCause()).contains("regions that are not 
online")
&& ie.getCause() instanceof MergeRegionException);
}{code}
+1

> Remove declaration of un-thrown exceptions and unused 
> setRegionStateBackToOpen() from MergeTableRegionsProcedure
> ----------------------------------------------------------------------------------------------------------------
>
>                 Key: HBASE-20055
>                 URL: https://issues.apache.org/jira/browse/HBASE-20055
>             Project: HBase
>          Issue Type: Improvement
>          Components: amv2
>    Affects Versions: 2.0.0-beta-1
>            Reporter: Umesh Agashe
>            Assignee: Umesh Agashe
>            Priority: Minor
>             Fix For: 2.0.0-beta-2
>
>         Attachments: hbase-20055.master.001.patch, 
> hbase-20055.master.001.patch, hbase-20055.master.002.patch
>
>
> A few methods declare exceptions in throws statement that are not thrown and 
> method setRegionStateBackToOpen() is not used in MergeTableRegionsProcedure.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to