[jira] [Commented] (HBASE-7211) Improve hbase ref guide for the testing part.

["Jeffrey Zhong (JIRA)" <ji...@apache.org>]

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

Jeffrey Zhong commented on HBASE-7211:
--------------------------------------

I happened to work on the issue a little bit and attached my patch to the issue. Inside the patch, there are following changes:

bq. => We could actually activate the localTests profile whenever -Dtest is used. If we do that, we can remove the reference from localTests in the doc.

Trigger localTest profile whenever "-Dtest" is specified so we can run individual tests like "mvm test –Dtest=MyTest" without specifying "localTests" profile. In addition, adding a note to let users pay attention on the number of "Tests run" because no error is reported when non-existent test cases are specified.

bq. => Integration tests are described TODO: POINTER_TO_INTEGRATION_TEST_SECTION

Instantiate "TODO: POINTER_TO_INTEGRATION_TEST_SECTION" with the right link

Change class name HBaseIntegrationTestingUtility to IntegrationTestingUtility because HBaseIntegrationTestingUtility doesn't exist at all.
                
> Improve hbase ref guide for the testing part.
> ---------------------------------------------
>
>                 Key: HBASE-7211
>                 URL: https://issues.apache.org/jira/browse/HBASE-7211
>             Project: HBase
>          Issue Type: Bug
>          Components: documentation
>    Affects Versions: 0.96.0
>            Reporter: nkeywal
>            Assignee: nkeywal
>            Priority: Minor
>         Attachments: hbase-7211-partial.patch
>
>
> Here is some stuff I saw. I will propose a fix in a week or so, please add the comment or issues you have in mind.
> ??15.6.1. Apache HBase Modules??
> => We should be able to use categories in all modules. The default should be small; but any test manipulating the time needs to be in a specific jvm (hence medium), so it's not always related to minicluster.
> ??15.6.3.6. hbasetests.sh??
> => We can remove this chapter, and the script
>  The script is not totally useless, but I think nobody actually uses it.
> => Add a chapter on flakiness.
> Some tests are, unfortunately, flaky. While there number decreases, we still have some. Rules are:
> - don't write flaky tests! :-)
> - small tests cannot be flaky, as it blocks other test execution. Corollary: if you have an issue with a small test, it's either your environment either a severe issue.
> - rerun the test a few time to validate, check the ports and file descriptors used.
> ??mvn test -P localTests -Dtest=MyTest??
> => We could actually activate the localTests profile whenever -Dtest is used. If we do that, we can remove the reference from localTests in the doc.
> ??mvn test -P runSmallTests?? ??mvn test -P runMediumTests??
> => I'm not sure it's actually used. We could remove them from the pom.xml (and the doc).
> ??The HBase build uses a patched version of the maven surefire plugin?? 
> => Hopefully, we will be able to remove this soon :-)
>  ??Integration tests are described TODO: POINTER_TO_INTEGRATION_TEST_SECTION??
>  => Should be documented

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira