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

Validation problems created when importing a SOAP-based wsdl

    XMLWordPrintable

Details

    • Bug
    • Resolution: Done
    • Major
    • 8.0
    • 8.0
    • Modeling
    • None

    Description

      Use the SOAP-based model importer to create source and view models based on a wsdl, already configured in a jboss 7 server.

      Both the source and view models are created successfully yet 6 errors are generated in the Problems view. See the attached screenshot.

      The US_States StateService example when this bug was encountered. This State Service was installed in jboss 7 so was modified accordingly. The new version can be downloaded from here.

      Investigation

      1. The 6 problems are generated from SqlTransformationMappingRootValidationRule (lines 589-596)
      2. The test (at line 589) checks the command's type, ie.
         if (cmdType == Command.TYPE_UPDATE_PROCEDURE) 
      3. The virtual procedure command instance is of the class CreateProcedureCommand
      • TransformationSqlHelper is responsible for creating the CreateProcedureCommand using the createVirtualProcCommmandForCommand() method
      • The CreateProcedureCommand command used to be CreateUpdateProcedureCommand but this was renamed in Teiid 8.0
      • The new command class had the API isUpdateProcedure() and setUpdateProcedure() methods removed
      • Commit #1e044a90 fixed the compilation errors resulting from this API change by
        • removing the setUpdateProcedure(false) call in TransformSqlHelper
        • removing the isUpdateProcedure() test and the end of line 589 in SqlTransformationMappingRootValidationRule

      Thus, originally the commands were instantiated and changed to not be 'update procedures'. This is enough to avoid the test and line 589.

      With the reduction of API and the compilation fixes, the commands incorrectly generate the validation errors.

      Attachments

        Activity

          People

            dflorian@redhat.com Daniel Florian (Inactive)
            parichar@redhat.com Paul Richardson
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: