Mailing-List: contact issues-help@hbase.apache.org; run by ezmlm
Precedence: bulk
Date: Fri, 19 Dec 2014 18:47:14 +0000 (UTC)
From: "Jonathan Hsieh (JIRA)" <jira@apache.org>
To: issues@hbase.apache.org
Message-ID: <JIRA.12762607.1418917586000.70869.1419014834184@Atlassian.JIRA>
In-Reply-To: <JIRA.12762607.1418917586000@Atlassian.JIRA>
References: <JIRA.12762607.1418917586000@Atlassian.JIRA>
 <JIRA.12762607.1418917586943@arcas>
Subject: [jira] [Commented] (HBASE-12718) Convert TestAcidGuarantees from a
 unit test to an integration test
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit


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

Jonathan Hsieh commented on HBASE-12718:
----------------------------------------

I considered it but lean towards only having it in IT.

1) The main functionality this move buys us is the ability to use ChaosMonkey while the test runs (since we derive off of IntegrationTestBase). To have it in both, I'd need to do some refactoring.  
2) The test is non deterministic as a unit test and does the whole spin up a minicluster while only using the public client apis.  if it fails it points out there a problem but doesn't necessarily tell us where.  Unit tests ideally tell us where.
3) It isn't like the test has gone away -- it still can be executed against a minicluster using mvn: 
{code}
mvn install -DskipTests
cd hbase-it
mvn verify -Dit.test=IntegrationTestAcidGuarantees
{code}

If those reasons are too weak, file an issue to refactor this and assign it to me.

> Convert TestAcidGuarantees from a unit test to an integration test
> ------------------------------------------------------------------
>
>                 Key: HBASE-12718
>                 URL: https://issues.apache.org/jira/browse/HBASE-12718
>             Project: HBase
>          Issue Type: Bug
>          Components: hbase, integration tests, test
>    Affects Versions: 1.0.0, 2.0.0
>            Reporter: Jonathan Hsieh
>            Assignee: Jonathan Hsieh
>             Fix For: 2.0.0, hbase-11339, 1.1.0
>
>         Attachments: hbase-12718.patch
>
>
> TestAcidGurantees has a main method so that it can be executed via the commandline.  In the past this was run and we'd use external tools to inject faults while it executed for an extended period of time.  We've had the IT framework we'd like to use the ChaosMonkey automation with it.


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