[
https://issues.apache.org/jira/browse/HBASE-21066?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16583595#comment-16583595
]
Xu Cang commented on HBASE-21066:
---------------------------------
I plan to propagate possible IOException to callers of this method and let
callers decide how to handle it.
> Improve isTableState() method to ensure caller gets correct info
> ----------------------------------------------------------------
>
> Key: HBASE-21066
> URL: https://issues.apache.org/jira/browse/HBASE-21066
> Project: HBase
> Issue Type: Improvement
> Affects Versions: 3.0.0
> Reporter: Xu Cang
> Assignee: Xu Cang
> Priority: Minor
>
>
> {code:java}
> public boolean isTableState(TableName tableName, TableState.State... states) {
> try {
> TableState tableState = getTableState(tableName);
> return tableState.isInStates(states);
> } catch (IOException e) {
> LOG.error("Unable to get table " + tableName + " state", e);
> // XXX: is it safe to just return false here?
> return false;
> }
> }
>
> {code}
>
> When cannot get table state, returning false is not always safe or correct.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
