Nikolay Samokhvalov wrote: > On 8/26/06, Peter Eisentraut <[EMAIL PROTECTED]> wrote: > > Bruce Momjian wrote: > > > Validation > > > ---------- > > > /contrib/xml2 has a function called xml_valid() that can be used in > > > a CHECK constraint to enforce that a field contains valid XML. It > > > does not support validation against a specific XML schema. > > > > Then this is not validation but only checking for well-formedness. The > > xml2 README says so, in fact. > > Exactly. contrib/xml2 mixes the term here, xml_valid() should be > another function, that takes two types of data - XML value and > corresponding XML schema - and validate the XML data. Actually, the > latest version of SQL/XML standard includes such a function > (XMLVALIDATE).
I understand, but do we want to break backward compatibility to rename it? We could create a xml_check, and keep xml_valid as a single-argument function, and implement schema-checks as a two-parameter function, but that seems odd too. > If you decide to include the mentioning about contrib/xml2 to docs, I > would suggest the patch for this module. The patch renames that > function to xml_check() and adds xml_array() (issue from the current > TODO). Or it's too late for 8.2? Hard to say. What does xml_array do? We are more lenient about /contrib additions after feature freeze, but it is pretty late. Aren't you working on updating the new XML syntax support in the backend? Are you done with that patch? > Also, I would add a little introduction to XML terms (from XML > standards) to this documentation section. OK, but which terms. I only see XML and XSLT, and I documented those on first mention in the newest version. -- Bruce Momjian [EMAIL PROTECTED] EnterpriseDB http://www.enterprisedb.com + If your life is a hard drive, Christ can be your backup. + ---------------------------(end of broadcast)--------------------------- TIP 5: don't forget to increase your free space map settings