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

Roman Shaposhnik commented on BIGTOP-614:
-----------------------------------------

Sujay, I agree with your comments above. Also, please include the following 
clean up into your new patch:
  # there doesn't seem to be any need for a shell class in TestHAFailover and 
TestKillNNFailover
  # the test shouldn't assume that it is executed under any specific user 
account (e.g. assumptions that it is root). If something requires root 
privileges it needs to be executed via a root shell. Something like shRoot = 
new Shell("/bin/bash -s", "root")
  # please get rid of System.out.println calls around shell calls. all shell 
tracing is done via log4j
  # it would be nice to get rid of the assertNotNull("JAVA_HOME has to be set 
to run this test", JAVA_HOME) and assertNotNull("HADOOP_HOME has to be set to 
run this test", HADOOP_HOME) in the HATestUtil
                
> Add HDFS HA tests
> -----------------
>
>                 Key: BIGTOP-614
>                 URL: https://issues.apache.org/jira/browse/BIGTOP-614
>             Project: Bigtop
>          Issue Type: Test
>          Components: Tests
>            Reporter: Stephen Chu
>            Assignee: Stephen Chu
>         Attachments: HATests.patch
>
>
> In this patch, I am adding 2 tests and a test util class.
> 1) TestHAFailover: Fails over between the NameNodes a configurable number of 
> times. Checks the service state of each NN after each failover to ensure that 
> the service state is correct.
> 2) TestKilLNNFailover: Kills the Active NN. Waits and fails over to the 
> standby NN. Waits and checks that the former standby is now active. Restarts 
> the killed NN and checks that it is now standby. Goes through this process a 
> configurable number of times.
> 3) HATestUtil: Helper methods to fail over and kill the NameNodes.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to