[ https://issues.apache.org/jira/browse/HADOOP-6879?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12912739#action_12912739 ]
Sharad Agarwal commented on HADOOP-6879: ---------------------------------------- Minor nits: 1. The method needs to be implemented or removed: {code} public void testExecuteCommandForRemoteHostNameUserCommandIdentityFilePortNumber() throws Exception { //TODO: Test goes here... } {code} 2. There shouldn't be any @author tags in the javadoc. Other than that patch looks fine. > Provide SSH based (Jsch) remote execution API for system tests > -------------------------------------------------------------- > > Key: HADOOP-6879 > URL: https://issues.apache.org/jira/browse/HADOOP-6879 > Project: Hadoop Common > Issue Type: Improvement > Components: build, test > Affects Versions: 0.22.0 > Reporter: Iyappan Srinivasan > Assignee: Iyappan Srinivasan > Attachments: 6879-ydist-security-patch.txt, HADOOP-6879.patch, > HADOOP-6879.patch, HADOOP-6879.patch, HADOOP-6879.patch, HADOOP-6879.patch, > HADOOP-6879.patch, HADOOP-6879.patch, HADOOP-6879.patch, > HADOOP-6879.y20.patch, HADOOP-6879.y20.patch, HADOOP-6879.y20.patch, > HADOOP-6879.y20.patch, HADOOP-6879.y20.patch > > > http://mvnrepository.com/ > com.jcraft ยป jsch > 0.1.42 version needs to be included in the build. This is needed to > facilitate implementation of some system (Herriot) testcases . > Please include this in ivy. > jsch is originally located in http://www.jcraft.com/jsch/ -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.