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

Andrey Stepachev commented on HBASE-11974:
------------------------------------------

yes. and even better, let's make it configurable, do we really need to ensure, 
that table is disabled.

so in general patch can be improved:
1. adding enum (or additional parameter) to prepare() to ensure, that table 
checked only once
2. add configuration parameter to minimise master RPC (say let it be general 
configuration to ensure, that different part of the codebase will try to not 
issue needless RPC to precious parts of the system, like masters)

> When a disabled table is scanned, NotServingRegionException is thrown instead 
> of TableNotEnabledException
> ---------------------------------------------------------------------------------------------------------
>
>                 Key: HBASE-11974
>                 URL: https://issues.apache.org/jira/browse/HBASE-11974
>             Project: HBase
>          Issue Type: Bug
>            Reporter: Ted Yu
>            Assignee: Ted Yu
>         Attachments: 11974-test.patch, 11974-v1.txt, 11974-v2.txt, 
> 11974-v3.txt, 11974-v4.txt, 11974-v5.txt, 11974-v6.txt
>
>
> When a disabled table is scanned, TableNotEnabledException should be thrown.
> However, currently NotServingRegionException is thrown.
> Thanks to Romil Choksi who discovered this problem.



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

Reply via email to