You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@hbase.apache.org by "nkeywal (JIRA)" <ji...@apache.org> on 2012/11/22 16:18:58 UTC

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

nkeywal created HBASE-7211:
------------------------------

             Summary: 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


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

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

Posted by "Enis Soztutar (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HBASE-7211?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13504221#comment-13504221 ] 

Enis Soztutar commented on HBASE-7211:
--------------------------------------

bq. => 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.
Right now, the only exception is hadoop-compat modules. 
See Elliot's comment: https://issues.apache.org/jira/browse/HBASE-7109?focusedCommentId=13503984&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-13503984
bq. => Add a chapter on flakiness.
+1.
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.
+1. -PlocalTets is not intuitive, and I was burned a couple of times by this. By having active in profile, this should be easy to do. 
bq. => Integration tests are described TODO: POINTER_TO_INTEGRATION_TEST_SECTION
We can just commit HBASE-6302. 

                
> 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
>
> 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

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

Posted by "stack (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HBASE-7211?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13504186#comment-13504186 ] 

stack commented on HBASE-7211:
------------------------------

These look like great changes N.

On IT stuff, thats HBASE-6302
                
> 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
>
> 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