<div dir="ltr">Hi,<div><br></div><div>I&#39;m starting to work with Teiid and would like to know what the procedure is for contributing.</div><div><br></div><div>I&#39;ve been working with the Swagger translator and have found a few bugs and also have an enhancement I&#39;d like to suggest.</div><div><br></div><div>Bugs</div><div><br></div><div>1) JsonSerializer.deserialize with an unnamed array<br></div><div><br></div><div>If a Rest endpoint is called that returns an unnamed array such as:</div><div><br></div><div><div>[</div><div>  {</div><div>    &quot;scheduleId&quot;: 10007,</div><div>    &quot;timeStamp&quot;: 1468000572174,</div><div>    &quot;value&quot;: <a href="tel:2813018112" value="+12813018112" target="_blank">2813018112</a></div><div>  },</div><div>  {</div><div>    &quot;scheduleId&quot;: 10007,</div><div>    &quot;timeStamp&quot;: 1468000602173,</div><div>    &quot;value&quot;: <a href="tel:2817523712" value="+12817523712" target="_blank">2817523712</a></div><div>  }</div><div>]</div></div><div><br></div><div>the JsonSerializer.deserialize method throws an EmptyStackException in the END_ARRAY case because the &#39;fieldName&#39; stack is empty.  A check for empty in the if clause fixes the problem:</div><div><br></div><div>  if (arrayLevel &gt; objectLevel &amp;&amp; !fieldName.empty())...<br></div><div><br></div><div>2) DDL generation</div><div>When the schema DDL is generated from the Swagger, all procedure parameters have their NullType set to NullType.No_Nulls.  Can this be set depending on the Swagger parameter &#39;required&#39; field?  For instance:</div><div><br></div><div><div>  boolean required = parameter.getRequired();</div><div>  pp.setNullType(required ? NullType.No_Nulls : NullType.Nullable);  </div></div><div><br></div><div><br></div><div>Enhancement idea:<br></div><div><br></div><div>REST apis that are not Swagger based could be used with the Swagger translator if a user could provide a swagger.json file they&#39;ve created in the Swagger Editor.  I&#39;ve added some changes to SwaggerMetadataProcessor that allow a user to specify an importer property, swaggerFilePath, that&#39;s used in the getSchema method to load the Swagger metadata from file.</div><div><br></div><div>Please let me know if you&#39;d like me to contribute these changes.</div><div><br></div><div>Thank,</div><div><br></div><div>Van</div></div>