[ 
https://issues.apache.org/jira/browse/HBASE-6914?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Elliott Clark updated HBASE-6914:
---------------------------------

    Attachment: HBASE-6914-2.patch

So TestHFileOutputFormat uses getRegionLocation to get cached region locations 
as a way to enumerate all of the regions.  This means that always throwing an 
error when trying to get a location would break the TestUtil.  So I tried a 
little different way.

This patch puts the thrown exception when trying to get the region location 
only when explicitly asking for the location to not come from cache.  This 
should happen when starting a scan(Scan's explicitly reload the region 
location) or after the first time a client gets a NotServingRegionExeption when 
trying to Get/Mutate on a disabled table.

It's a pretty delicate balance so I added some comments.
                
> Scans/Gets/Mutations don't give a good error if the table is disabled.
> ----------------------------------------------------------------------
>
>                 Key: HBASE-6914
>                 URL: https://issues.apache.org/jira/browse/HBASE-6914
>             Project: HBase
>          Issue Type: Improvement
>            Reporter: Elliott Clark
>            Assignee: Elliott Clark
>             Fix For: 0.92.3, 0.94.3, 0.96.0
>
>         Attachments: HBASE-6914-0.patch, HBASE-6914-1.patch, 
> HBASE-6914-2.patch
>
>
> Scan a table that is disabled will have the client retry multiple times and 
> then will error out with NotServingRegionException.  If the table is disabled 
> there's no need to re-try and the message should be more explicit.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to