pg.ddx.io  pgsql-docs@postgresql.org mailing list archive  
help / color / mirror / Atom feed
XMLTABLE default namespace in docs
2+ messages / 2 participants
[nested] [flat]

Commits from this thread:
  2018-06-21 c7048977a7 b742010890  Disclaim support for default namespace in XMLTABLE
* XMLTABLE default namespace in docs
@ 2017-09-11 15:25  Daniel Gustafsson <daniel@yesql.se>
  0 siblings, 1 reply; 2+ messages in thread

From: Daniel Gustafsson @ 2017-09-11 15:25 UTC (permalink / raw)
  To: pgsql-docs; Pavel Stehule <pavel.stehule@gmail.com>

When reviewing Pavels patch for default namespace in XMLTABLE, I came across
this sentence in the devel (and 10beta) docs in subsection 9.14.3.3. xmltable:

	The following example illustrates how the XMLNAMESPACES clause can be
	used to specify the default namespace, and a list of additional
	namespaces used in the XML document as well as in the XPath
	expressions:

That seems odd since we don’t support default namespaces (hence the patch for
adding it).  Am I reading it wrong, or shouldn’t we apply something like the
attached for now?

cheers ./daniel

Attachments:

  xmltable-def-ns.patch (application/octet-stream, 0B) download

^ permalink  raw  reply  [nested|flat] 2+ messages in thread

* Re: [DOCS] XMLTABLE default namespace in docs
@ 2018-06-21 21:03  Alvaro Herrera <alvherre@2ndquadrant.com>
  parent: Daniel Gustafsson <daniel@yesql.se>
  0 siblings, 0 replies; 2+ messages in thread

From: Alvaro Herrera @ 2018-06-21 21:03 UTC (permalink / raw)
  To: Daniel Gustafsson <daniel@yesql.se>; +Cc: pgsql-docs; Pavel Stehule <pavel.stehule@gmail.com>

On 2017-Sep-11, Daniel Gustafsson wrote:

> When reviewing Pavels patch for default namespace in XMLTABLE, I came across
> this sentence in the devel (and 10beta) docs in subsection 9.14.3.3. xmltable:
> 
> 	The following example illustrates how the XMLNAMESPACES clause can be
> 	used to specify the default namespace, and a list of additional
> 	namespaces used in the XML document as well as in the XPath
> 	expressions:
> 
> That seems odd since we don’t support default namespaces (hence the patch for
> adding it).  Am I reading it wrong, or shouldn’t we apply something like the
> attached for now?

You're right, we should -- thanks for noticing.  I have pushed it now.

-- 
Álvaro Herrera                https://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services




^ permalink  raw  reply  [nested|flat] 2+ messages in thread


end of thread, other threads:[~2018-06-21 21:03 UTC | newest]

Thread overview: 2+ messages (download: mbox mbox.gz follow: Atom feed)
-- links below jump to the message on this page --
2017-09-11 15:25 XMLTABLE default namespace in docs Daniel Gustafsson <daniel@yesql.se>
2018-06-21 21:03 ` Alvaro Herrera <alvherre@2ndquadrant.com>

This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox