You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@hbase.apache.org by "Chia-Ping Tsai (JIRA)" <ji...@apache.org> on 2018/02/27 08:26:00 UTC

[jira] [Updated] (HBASE-20097) Merge TableDescriptors#getAll and TableDescriptors#getAllDescriptors into one

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

Chia-Ping Tsai updated HBASE-20097:
-----------------------------------
    Summary: Merge TableDescriptors#getAll and TableDescriptors#getAllDescriptors into one  (was: Remove TableDescriptors#getAll since it just clone the returned object from TableDescriptors#getAllDescriptors)

> Merge TableDescriptors#getAll and TableDescriptors#getAllDescriptors into one
> -----------------------------------------------------------------------------
>
>                 Key: HBASE-20097
>                 URL: https://issues.apache.org/jira/browse/HBASE-20097
>             Project: HBase
>          Issue Type: Task
>            Reporter: Chia-Ping Tsai
>            Assignee: Chia-Ping Tsai
>            Priority: Minor
>             Fix For: 2.0.0
>
>
> {code:java}
> @Override
> public Map<String, TableDescriptor> getAll() throws IOException {
>   Map<String, TableDescriptor> htds = new TreeMap<>();
>   Map<String, TableDescriptor> allDescriptors = getAllDescriptors();
>   for (Map.Entry<String, TableDescriptor> entry : allDescriptors
>       .entrySet()) {
>     htds.put(entry.getKey(), entry.getValue());
>   }
>   return htds;
> }{code}
> The returned map from #getAllDescriptors isn't a inner object so doing the clone is meaningless.



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