hbase-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Enis Soztutar (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (HBASE-6241) HBaseCluster interface for interacting with the cluster from system tests
Date Sat, 30 Jun 2012 00:02:45 GMT

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

Enis Soztutar commented on HBASE-6241:

Forgot to mention that patch at RB addresses all of the Ted's comments above, except: 
bq. If no enum argument is planned to be added, the method can be renamed as waitOnMasterToStop().

renamed methods like waitForMasterToStop()

bq. +  public void restoreInitialState() throws IOException {
bq. How about restoring to one of the ClusterStatus previously captured by getClusterStatus()
? (future enhancement)
bq. I think getInitialClusterStatus() and getClusterStatus() can be unified into one method
with enum indicating the type of status.

Now, we have restoreInitialStatus(), and restoreClusterStatus(ClusterStatus desiredStatus).
The former just calls the latter with the result from getInitialClusterStatus()

bq. For HBaseTestingUtility.java:
+    } catch (Exception ex) {
+      throw new RuntimeException(ex);
+    }
Moved the exception catching an throwing into the constructor. 
> HBaseCluster interface for interacting with the cluster from system tests 
> --------------------------------------------------------------------------
>                 Key: HBASE-6241
>                 URL: https://issues.apache.org/jira/browse/HBASE-6241
>             Project: HBase
>          Issue Type: Sub-task
>            Reporter: Enis Soztutar
>            Assignee: Enis Soztutar
>         Attachments: HBASE-6241_v0.2.patch, HBASE-6241_v1.patch
> We need to abstract away the cluster interactions for system tests running on actual
> MiniHBaseCluster and RealHBaseCluster should both implement this interface, and system
tests should work with both.
> I'll split Devaraj's patch in HBASE-6053 for the initial version. 

This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira


View raw message