Stefano, I have merged the PR. A system property should be fine for now. 
We can either make it the default or a Hint in a later release.

Kind regards,
Ben.

On 25/08/15 19:41, Stefano Costa wrote:
> Thanks Ben for your prompt and valuable feedback!
>
> I updated the PR according to your suggestion.
>
> About the use of a system property, I'm still unsure whether in the end
> we'll want to remove it: if the performance overhead proves negligible, we
> would make it default, but if the performance overhead proves significant,
> there might still be situations where enabling it could be the only way to
> make sure all definitions from external schemas are correctly imported
> (e.g. a schema imports 2 other schemas, but contains elements / types
> related only to the first schema: I believe in this case the second schema
> will not be imported, unless a forced import is triggered).
>
> Should I turn it into a Hint?

-- 
Ben Caradoc-Davies <b...@transient.nz>
Director
Transient Software Limited <http://transient.nz/>
New Zealand

------------------------------------------------------------------------------
_______________________________________________
GeoTools-Devel mailing list
GeoTools-Devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geotools-devel

Reply via email to