airavata-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF GitHub Bot (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (AIRAVATA-2163) optimize credential store data models
Date Tue, 13 Dec 2016 22:15:58 GMT

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

ASF GitHub Bot commented on AIRAVATA-2163:
------------------------------------------

Github user asfgit closed the pull request at:

    https://github.com/apache/airavata-php-gateway/pull/53


> optimize credential store data models
> -------------------------------------
>
>                 Key: AIRAVATA-2163
>                 URL: https://issues.apache.org/jira/browse/AIRAVATA-2163
>             Project: Airavata
>          Issue Type: Improvement
>          Components: Airavata API
>         Environment: Development
>            Reporter: Anuj Bhandar
>            Assignee: Anuj Bhandar
>              Labels: GATask
>             Fix For: 17
>
>
> This improvement is based on the following Pull request : https://github.com/apache/airavata/pull/54.
> **Improvements agreed upon :** 
> Followings are few suggestions to improve your pull request.
>     * We can do this in a better way by adding generic API method(to AiravataAPIServer)
like "getAllCredentialSummary" instead of adding API method for each credential types eg:
"getAllGatewaySSHPubKeysSummary" which will return all the credential summary for that particular
gatewayId. You may be able to remove few redundant methods from Airavata API server.
>     * Let's introduced one credential summary struct to all credential types, how we
differentiate is we have credential type enum field( SSH, PASSWD, CERT etc ) in the credential
summary thrift struct.
>      * Move credential data model thrift file to data model directory and merge credential
summary thrift file with it.
>      * Credential stubs still have date fields in generated thrift files, not because
your changes, but we need to fix it so remove it.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Mime
View raw message