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

Heng Chen updated HBASE-14203:
------------------------------
    Attachment: HBASE-14203_v3.patch

{quote}
Don't synchronize on a non-final field. Additionally I'm pretty sure that most 
of the tests are failing as they assume that getAdmin will create a new 
HBaseAdmin.
{quote}

Thanks for your replay!

Testcase failed is due to NPE when {{synchronized(admin)}},  I use another 
final field {{adminLock}} to sync as you said. 

The patch is updated!

> remove duplicate code getTableDescriptor in HTable
> --------------------------------------------------
>
>                 Key: HBASE-14203
>                 URL: https://issues.apache.org/jira/browse/HBASE-14203
>             Project: HBase
>          Issue Type: Improvement
>            Reporter: Heng Chen
>            Priority: Trivial
>         Attachments: HBASE-14203.patch, HBASE-14203_v2.patch, 
> HBASE-14203_v3.patch
>
>
> As TODO in comment said, 
> {{HTable.getTableDescriptor}} is same as {{HAdmin.getTableDescriptor}}. 
> remove the duplicate code.



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

Reply via email to