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

Mike Drob edited comment on HBASE-20660 at 5/31/18 1:32 AM:
------------------------------------------------------------

The Optional.get() there can throw, do we need to check isPresent() or use one 
of the orElse*() methods?

Edit: This would have been a problem before also, so it shouldn't block the 
patch. I'm not familiar enough with this code to make specific review, so I'm 
approaching it from general principles.


was (Author: mdrob):
The Optional.get() there can throw, do we need to check isPresent() or use one 
of the orElse*() methods?

> Reopen regions using ReopenTableRegionsProcedure
> ------------------------------------------------
>
>                 Key: HBASE-20660
>                 URL: https://issues.apache.org/jira/browse/HBASE-20660
>             Project: HBase
>          Issue Type: Sub-task
>            Reporter: Duo Zhang
>            Assignee: Duo Zhang
>            Priority: Major
>             Fix For: HBASE-19064
>
>         Attachments: HBASE-20660-HBASE-19064.patch
>
>




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

Reply via email to