Uploaded image for project: 'Teiid Designer'
  1. Teiid Designer
  2. TEIIDDES-2163

REST based importer design is wrong

    XMLWordPrintable

Details

    • Bug
    • Resolution: Duplicate
    • Critical
    • None
    • 8.3.2
    • Import/Export
    • None

    Description

      OK, I tried the REST based import wizard for the first time today since none of WSDL based solutions worked for me for those issues see TEIIDDES-2162

      I believe the REST based importer is designed to generate wrong artifacts. It was designed same as XML file, where a given XML file, Teiid designer aids the user to come up with a "view". The difference between a XML file and REST based service is, a

      • REST service will have input parameters. It is the input parameters that makes it a service, otherwise it just a remote URL.
      • REST Service can have optional paths

      Take for example WSDL based SOAP importer also gets XML as result, but it is designed to produce a procedure to capture the input parameters and return tabular result. REST based service MUST be doing exact same thing.

      Teiid invokeHTTP does work with relative and absolute URLs, Teiid even has "querystring" function https://docs.jboss.org/author/display/TEIID/BNF+for+SQL+Grammar#BNFforSQLGrammar-queryString

      to help build the query parameter part of the URL, then you just need to concat it with the URL to make the whole.

      Then we also need give a way to build optional path, based on input parameter to procedure, so in reality a parameter can be part of path or parameter

      IMO the way we have does not serve any real usecase, like mine where I was trying to read a quotes from REST service

      http://ws.cdyne.com/delayedstockquote/delayedstockquote.asmx/GetQuickQuote?StockSymbol=?&LicenseKey=?

      Attachments

        Issue Links

          Activity

            People

              Unassigned Unassigned
              rhn-engineering-rareddy Ramesh Reddy
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: