[teiid-issues] [JBoss JIRA] (TEIID-2729) Dynamic VDB xml requires sibling elements to be in a certain order

Steven Hawkins (JIRA) issues at jboss.org
Wed Sep 23 16:49:00 EDT 2015


    [ https://issues.jboss.org/browse/TEIID-2729?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13111927#comment-13111927 ] 

Steven Hawkins commented on TEIID-2729:
---------------------------------------

> IMO, validation out weighs the convenience. Granted we can remove validation, then we need to do certain tasks in code, which seems counter intuitive.

I don't quite follow you. As long as the parsing correctly validates (which is already does to a certain extent - only expected child elements can be processed), then having the extra xsd validation is simply an extra step.

> May be can put choice block around property, duplicate it before and after like

Can you take a wack at the xsd approach?  If it doesn't work, I'll just take a pass through the parsing logic and see how cumbersome it would be to have additional validation.

> Dynamic VDB xml requires sibling elements to be in a certain order
> ------------------------------------------------------------------
>
>                 Key: TEIID-2729
>                 URL: https://issues.jboss.org/browse/TEIID-2729
>             Project: Teiid
>          Issue Type: Enhancement
>          Components: AdminApi
>    Affects Versions: 8.5
>            Reporter: Mark Drilling
>            Assignee: Steven Hawkins
>             Fix For: 8.7
>
>
> In my *-vdb.xml, I have a model defined per following snippet:
> <model name="myModel" type="PHYSICAL" visible="true">
>     <source name="myModel" translator-name="hive" connection-jndi-name="Hive12Src" />
>     <property name="trimColumnNames" value="true" />
> </model>
> The vdb.xml failed to parse, so I moved the <property> element ahead of <source> and then it worked.
> I don't think element ordering should matter in this case?



--
This message was sent by Atlassian JIRA
(v6.4.11#64026)


More information about the teiid-issues mailing list