[
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
