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

stack commented on HBASE-12518:
-------------------------------

This is more of the same cleanup.  Will commit tomorrow.  Review in meantime 
would be grand. Thanks.

> Task 4 polish. Remove CM#{get,delete}Connection
> -----------------------------------------------
>
>                 Key: HBASE-12518
>                 URL: https://issues.apache.org/jira/browse/HBASE-12518
>             Project: HBase
>          Issue Type: Sub-task
>          Components: test
>    Affects Versions: 2.0.0, 0.99.2
>            Reporter: stack
>            Assignee: stack
>             Fix For: 2.0.0, 0.99.2
>
>         Attachments: 12518.txt
>
>
> HBASE-12471 purged get/delete Connection from under src/main/java. Lets 
> finish it up by doing same over in src/test since not that many places left.  
> Depends on HBASE-12404 going in first so can clean out references in 
> templates:
> {code}
> ./hbase-client/src/main/java/org/apache/hadoop/hbase/client/ConnectionManager.java:
>         ConnectionManager.deleteConnection(connectionKey, true);
> ./hbase-client/src/main/java/org/apache/hadoop/hbase/client/ConnectionManager.java:
>   public static void deleteConnection(Configuration conf) {
> ./hbase-client/src/main/java/org/apache/hadoop/hbase/client/ConnectionManager.java:
>     deleteConnection(new HConnectionKey(conf), false);
> ./hbase-client/src/main/java/org/apache/hadoop/hbase/client/ConnectionManager.java:
>     deleteConnection(connection, true);
> ./hbase-client/src/main/java/org/apache/hadoop/hbase/client/ConnectionManager.java:
>         deleteConnection(connectionKey, staleConnection);
> ./hbase-client/src/main/java/org/apache/hadoop/hbase/client/ConnectionManager.java:
>   private static void deleteConnection(HConnection connection, boolean 
> staleConnection) {
> ./hbase-client/src/main/java/org/apache/hadoop/hbase/client/ConnectionManager.java:
>           deleteConnection(e.getKey(), staleConnection);
> ./hbase-client/src/main/java/org/apache/hadoop/hbase/client/ConnectionManager.java:
>   private static void deleteConnection(HConnectionKey connectionKey, boolean 
> staleConnection) {
> ./hbase-client/src/main/java/org/apache/hadoop/hbase/client/ConnectionManager.java:
>           ConnectionManager.deleteConnection(this, false);
> ./hbase-client/src/main/java/org/apache/hadoop/hbase/client/HConnectionManager.java:
>  * Currently, clients cleanup by calling {@link 
> #deleteConnection(Configuration)}. This will
> ./hbase-client/src/main/java/org/apache/hadoop/hbase/client/HConnectionManager.java:
>  * HConnectionManager.deleteConnection(conf, true);
> ./hbase-client/src/main/java/org/apache/hadoop/hbase/client/HConnectionManager.java:
>   public static void deleteConnection(Configuration conf) {
> ./hbase-client/src/main/java/org/apache/hadoop/hbase/client/HConnectionManager.java:
>     ConnectionManager.deleteConnection(conf);
> ./hbase-server/src/main/jamon/org/apache/hadoop/hbase/tmpl/master/MasterStatusTmpl.jamon:
>    HConnectionManager.deleteConnection(admin.getConfiguration());
> ./hbase-server/src/main/resources/hbase-webapps/master/snapshot.jsp:HConnectionManager.deleteConnection(hbadmin.getConfiguration());
> ./hbase-server/src/main/resources/hbase-webapps/master/table.jsp:HConnectionManager.deleteConnection(hbadmin.getConfiguration());
> ./hbase-server/src/test/java/org/apache/hadoop/hbase/client/HConnectionTestingUtility.java:
>    * {@link HConnectionManager#deleteConnection(Configuration)} else it
> ./hbase-server/src/test/java/org/apache/hadoop/hbase/client/HConnectionTestingUtility.java:
>    * {@link HConnectionManager#deleteConnection(Configuration)} else it
> ./hbase-server/src/test/java/org/apache/hadoop/hbase/client/HConnectionTestingUtility.java:
>    * {@link HConnectionManager#deleteConnection(Configuration)}
> ./hbase-server/src/test/java/org/apache/hadoop/hbase/client/HConnectionTestingUtility.java:
>    * {@link HConnectionManager#deleteConnection(Configuration)} else it
> ./hbase-server/src/test/java/org/apache/hadoop/hbase/client/TestFromClientSide.java:
>      HConnectionManager.deleteConnection(newConfig);
> ./hbase-server/src/test/java/org/apache/hadoop/hbase/client/TestFromClientSide.java:
>      HConnectionManager.deleteConnection(newConfig2);
> ./hbase-server/src/test/java/org/apache/hadoop/hbase/client/TestHBaseAdminNoCluster.java.orig:
>       if (connection != 
> null)HConnectionManager.deleteConnection(configuration);
> ./hbase-server/src/test/java/org/apache/hadoop/hbase/client/TestHCM.java:     
>    HConnectionManager.deleteConnection(c.getConfiguration());
> ./hbase-server/src/test/java/org/apache/hadoop/hbase/master/TestCatalogJanitor.java:
>         
> HConnectionManager.deleteConnection(this.connection.getConfiguration());
> ./hbase-server/src/test/java/org/apache/hadoop/hbase/TestMetaTableAccessorNoCluster.java:
>       HConnectionManager.deleteConnection(UTIL.getConfiguration());
> ./hbase-server/src/test/java/org/apache/hadoop/hbase/TestMetaTableLocator.java:
>     HConnectionManager.deleteConnection(UTIL.getConfiguration());
> ./hbase-server/src/test/java/org/apache/hadoop/hbase/TestMetaTableLocator.java:
>    * {@link 
> HConnectionManager#deleteConnection(org.apache.hadoop.conf.Configuration)}
> ./hbase-server/src/test/java/org/apache/hadoop/hbase/util/hbck/OfflineMetaRebuildTestCore.java:
>     HConnectionManager.deleteConnection(conf);
> ./hbase-server/src/test/java/org/apache/hadoop/hbase/util/hbck/TestOfflineMetaRebuildBase.java:
>     HConnectionManager.deleteConnection(conf);
> {code}



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

Reply via email to