[
https://issues.apache.org/jira/browse/AIRAVATA-2163?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15742618#comment-15742618
]
ASF GitHub Bot commented on AIRAVATA-2163:
------------------------------------------
GitHub user anujbhan opened a pull request:
https://github.com/apache/airavata/pull/83
Credential store data model optimization
https://issues.apache.org/jira/browse/AIRAVATA-2163
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/anujbhan/airavata cred-opt
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/airavata/pull/83.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #83
----
commit 122fb8da06408b1f203006e88a97e923be4fbc2a
Author: Anuj Bhandar <[email protected]>
Date: 2016-12-12T09:18:06Z
migrating cred data models to airavata datamodel directory
commit c1b696badd3ccfa686cdb21ab2da0012ec7dd354
Author: Anuj Bhandar <[email protected]>
Date: 2016-12-12T16:58:57Z
introducing summary type, dependencies and code clean up
----
> 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)
