New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Feature/ftp transport - FTP Transport protocol PR #13
Conversation
You have missed including the dependency for your FTP-transport under mft-agent/pom.xml |
I have added it! |
...er/src/main/java/org/apache/airavata/mft/resource/server/backend/sql/SQLResourceBackend.java
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code looks good. Minor change requests. Add license headers to new files
...c/main/java/org/apache/airavata/mft/resource/server/backend/sql/entity/FTPStorageEntity.java
Show resolved
Hide resolved
@@ -0,0 +1,57 @@ | |||
package org.apache.airavata.mft.resource.server.backend.sql.entity; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
License headers?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Agreed! I will add the license headers
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done! Pushed the changes
Looks good to me. Great work. Thanks |
@@ -55,4 +55,9 @@ | |||
public DropboxSecret createDropboxSecret(DropboxSecretCreateRequest request) throws Exception; | |||
public boolean updateDropboxSecret(DropboxSecretUpdateRequest request) throws Exception; | |||
public boolean deleteDropboxSecret(DropboxSecretDeleteRequest request) throws Exception; | |||
|
|||
Optional<FTPSecret> getFTPSecret(FTPSecretGetRequest request) throws Exception; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@gkiran292 @DImuthuUpe
For keeping it uniform, I guess adding a public modifier to each of these declarations would be better.
FTP transport changes.
@DImuthuUpe Please review this.