Hi, I want to bring the importance of maintaining stable HBase build to our attention. A stable HBase build is important, not just for the next release but also for authors of the pending patches to verify the correctness of their work.
At some time on Thursday (Sept 22nd) 0.90, 0.92 and TRUNK builds were all blue. Now they're all red. I don't mind fixing Jenkins build. But if we collectively adopt some good practice, it would be easier to achieve the goal of having stable builds. For contributors, I understand that it takes so much time to run whole test suite that he/she may not have the luxury of doing this - Apache Jenkins wouldn't do it when you press Submit Patch button. If this is the case (let's call it scenario A), please use Eclipse (or other tool) to identify tests that exercise the classes/methods in your patch and run them. Also clearly state what tests you ran in the JIRA. If you have a Linux box where you can run whole test suite, it would be nice to utilize such resource and run whole suite. Then please state this fact on the JIRA as well. Considering Todd's suggestion of holding off commit for 24 hours after code review, 2 hour test run isn't that long. Sometimes you may see the following (from 0.92 build 18): Tests run: 1004, Failures: 0, Errors: 0, Skipped: 21 [INFO] ------------------------------------------------------------------------ [INFO] BUILD FAILURE [INFO] ------------------------------------------------------------------------ [INFO] Total time: 1:51:41.797s You should examine the test summary above these lines and find out which test(s) hung. For this case it was TestMasterFailover: Running org.apache.hadoop.hbase.master.TestMasterFailover Running org.apache.hadoop.hbase.master.TestMasterRestartAfterDisablingTable Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 32.265 sec I think a script should be developed that parses test output and identify hanging test(s). For scenario A, I hope committer would run test suite. The net effect would be a statement on the JIRA, saying all tests passed. Your comments/suggestions are welcome.
