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