[teiid-issues] [JBoss JIRA] (TEIID-3632) Performance issue retrieving data source and translator definitions through the AdminFactory

[Ramesh Reddy (JIRA)]

     [ https://issues.jboss.org/browse/TEIID-3632?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Ramesh Reddy resolved TEIID-3632.
---------------------------------
        Labels: Beta2  (was: )
    Resolution: Done


Made couple more changes

- For the calls getInstalledResourceAdaptorNames(), getConnectionFactoryNames(), getDeployedResourceAdaptorNames() introducing a time based expiring cache object that lives for 5 secs. So, any admin calls made inside that 5 secs use the cached objects rather then re-issuing the query. I do not go any lengths to remove these objects after 5 secs, rather I mark them as expired. If they are expired another set of fresh CLI calls will be made transparently when next time they are accessed. 

- Introduced "ALL" TranslatorPropertyType, using which one can ask for all the IMPORT, OVERRIDE, EXTENSION properties on the  getTranslatorPropertyDefinitions() method. The returned properties will have a new attribute called "category" which will define the type of the property IMPORT, OVERRIDE, EXTENSION

> Performance issue retrieving data source and translator definitions through the AdminFactory
> --------------------------------------------------------------------------------------------
>
>                 Key: TEIID-3632
>                 URL: https://issues.jboss.org/browse/TEIID-3632
>             Project: Teiid
>          Issue Type: Bug
>          Components: AdminApi
>            Reporter: Barry LaFond
>            Assignee: Ramesh Reddy
>              Labels: Beta2
>             Fix For: 8.12
>
>         Attachments: server_refresh_cli_log.txt
>
>
> Based on the issue described in  TEIIDDES-2592, we discovered that connecting to a remote server that contained ~ 40 data sources required > 10 minutes to refresh the Server view in Teiid Designer.  After some debugging it appeared that some of the current behavior in AdminFactory results multiple redundant CLI calls to retrieve intermediate information to compile even a simple list of data source names.
> In Teiid Designer, we modified our forked Admin8Factory to define a few cached sets of data that get cleared if data is changed by our ExecutionAdmin class for any created or deleted data sources.
> Note that Designer's Server management paradigm includes the requirement to view the DS/Translator/VDB contents on the server.
> So I'm assuming that there could be some benefit if Teiid AdminFactory adapted a similar approach to improving the performance/behavior of the internal methods of this class. 
> see: https://github.com/Teiid-Designer/teiid-designer/blob/master/plugins/teiid/org.teiid.runtime.client/src/org/teiid/runtime/client/admin/v8/Admin8Factory.java for details



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)