Skip to content

P
Postgres FD Implementation
Commit b7f0be9a authored by Alvaro Herrera's avatar Alvaro Herrera
Browse files
Options

Accept TEXT and CDATA nodes in XMLTABLE's column_expression. 

Column expressions that match TEXT or CDATA nodes must return the
contents of the nodes themselves, not the content of non-existing
children (i.e. the empty string).

Author: Markus Winand
Reported-by: Markus Winand
Reviewed-by: Álvaro Herrera
Discussion: https://postgr.es/m/0684A598-002C-42A2-AE12-F024A324EAE4@winand.at
parent 3adcad45
Hide whitespace changes
Inline Side-by-side
Showing with 28 additions and 24 deletions
src/backend/utils/adt/xml.c
View file @ b7f0be9a
...@@ -4508,11 +4508,21 @@ XmlTableGetValue(TableFuncScanState *state, int colnum, ...@@ -4508,11 +4508,21 @@ XmlTableGetValue(TableFuncScanState *state, int colnum,
else if (count == 1) else if (count == 1)
{ {
xmlChar *str; xmlChar *str;
xmlNodePtr node;
str = xmlNodeListGetString(xtCxt->doc, /*
xpathobj->nodesetval->nodeTab[0]->xmlChildrenNode, * Most nodes (elements and even attributes) store their data
1); * in children nodes. If they don't have children nodes, it
* means that they are empty (e.g. <element/>). Text nodes and
* CDATA sections are an exception: they don't have children
* but have content in the Text/CDATA node itself.
*/
node = xpathobj->nodesetval->nodeTab[0];
if (node->type != XML_CDATA_SECTION_NODE &&
node->type != XML_TEXT_NODE)
node = node->xmlChildrenNode;
str = xmlNodeListGetString(xtCxt->doc, node, 1);
if (str != NULL) if (str != NULL)
{ {
PG_TRY(); PG_TRY();
...@@ -4529,13 +4539,7 @@ XmlTableGetValue(TableFuncScanState *state, int colnum, ...@@ -4529,13 +4539,7 @@ XmlTableGetValue(TableFuncScanState *state, int colnum,
} }
else else
{ {
/* /* Ensure mapping of empty tags to PostgreSQL values. */
* This line ensure mapping of empty tags to PostgreSQL
* value. Usually we would to map a empty tag to empty
* string. But this mapping can create empty string when
* user doesn't expect it - when empty tag is enforced by
* libxml2 - when user uses a text() function for example.
*/
cstr = ""; cstr = "";
} }
} }
......
src/test/regress/expected/xml.out
View file @ b7f0be9a
...@@ -1024,7 +1024,7 @@ SELECT xmltable.* ...@@ -1024,7 +1024,7 @@ SELECT xmltable.*
PASSING data PASSING data
COLUMNS id int PATH '@id', COLUMNS id int PATH '@id',
_id FOR ORDINALITY, _id FOR ORDINALITY,
country_name text PATH 'COUNTRY_NAME' NOT NULL, country_name text PATH 'COUNTRY_NAME/text()' NOT NULL,
country_id text PATH 'COUNTRY_ID', country_id text PATH 'COUNTRY_ID',
region_id int PATH 'REGION_ID', region_id int PATH 'REGION_ID',
size float PATH 'SIZE', size float PATH 'SIZE',
...@@ -1046,7 +1046,7 @@ CREATE VIEW xmltableview1 AS SELECT xmltable.* ...@@ -1046,7 +1046,7 @@ CREATE VIEW xmltableview1 AS SELECT xmltable.*
PASSING data PASSING data
COLUMNS id int PATH '@id', COLUMNS id int PATH '@id',
_id FOR ORDINALITY, _id FOR ORDINALITY,
country_name text PATH 'COUNTRY_NAME' NOT NULL, country_name text PATH 'COUNTRY_NAME/text()' NOT NULL,
country_id text PATH 'COUNTRY_ID', country_id text PATH 'COUNTRY_ID',
region_id int PATH 'REGION_ID', region_id int PATH 'REGION_ID',
size float PATH 'SIZE', size float PATH 'SIZE',
...@@ -1075,7 +1075,7 @@ CREATE OR REPLACE VIEW public.xmltableview1 AS ...@@ -1075,7 +1075,7 @@ CREATE OR REPLACE VIEW public.xmltableview1 AS
"xmltable".premier_name "xmltable".premier_name
FROM ( SELECT xmldata.data FROM ( SELECT xmldata.data
FROM xmldata) x, FROM xmldata) x,
LATERAL XMLTABLE(('/ROWS/ROW'::text) PASSING (x.data) COLUMNS id integer PATH ('@id'::text), _id FOR ORDINALITY, country_name text PATH ('COUNTRY_NAME'::text) NOT NULL, country_id text PATH ('COUNTRY_ID'::text), region_id integer PATH ('REGION_ID'::text), size double precision PATH ('SIZE'::text), unit text PATH ('SIZE/@unit'::text), premier_name text DEFAULT ('not specified'::text) PATH ('PREMIER_NAME'::text)) LATERAL XMLTABLE(('/ROWS/ROW'::text) PASSING (x.data) COLUMNS id integer PATH ('@id'::text), _id FOR ORDINALITY, country_name text PATH ('COUNTRY_NAME/text()'::text) NOT NULL, country_id text PATH ('COUNTRY_ID'::text), region_id integer PATH ('REGION_ID'::text), size double precision PATH ('SIZE'::text), unit text PATH ('SIZE/@unit'::text), premier_name text DEFAULT ('not specified'::text) PATH ('PREMIER_NAME'::text))
EXPLAIN (COSTS OFF) SELECT * FROM xmltableview1; EXPLAIN (COSTS OFF) SELECT * FROM xmltableview1;
QUERY PLAN QUERY PLAN
----------------------------------------- -----------------------------------------
...@@ -1085,15 +1085,15 @@ EXPLAIN (COSTS OFF) SELECT * FROM xmltableview1; ...@@ -1085,15 +1085,15 @@ EXPLAIN (COSTS OFF) SELECT * FROM xmltableview1;
(3 rows) (3 rows)
EXPLAIN (COSTS OFF, VERBOSE) SELECT * FROM xmltableview1; EXPLAIN (COSTS OFF, VERBOSE) SELECT * FROM xmltableview1;
QUERY PLAN QUERY PLAN
----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- ------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
Nested Loop Nested Loop
Output: "xmltable".id, "xmltable"._id, "xmltable".country_name, "xmltable".country_id, "xmltable".region_id, "xmltable".size, "xmltable".unit, "xmltable".premier_name Output: "xmltable".id, "xmltable"._id, "xmltable".country_name, "xmltable".country_id, "xmltable".region_id, "xmltable".size, "xmltable".unit, "xmltable".premier_name
-> Seq Scan on public.xmldata -> Seq Scan on public.xmldata
Output: xmldata.data Output: xmldata.data
-> Table Function Scan on "xmltable" -> Table Function Scan on "xmltable"
Output: "xmltable".id, "xmltable"._id, "xmltable".country_name, "xmltable".country_id, "xmltable".region_id, "xmltable".size, "xmltable".unit, "xmltable".premier_name Output: "xmltable".id, "xmltable"._id, "xmltable".country_name, "xmltable".country_id, "xmltable".region_id, "xmltable".size, "xmltable".unit, "xmltable".premier_name
Table Function Call: XMLTABLE(('/ROWS/ROW'::text) PASSING (xmldata.data) COLUMNS id integer PATH ('@id'::text), _id FOR ORDINALITY, country_name text PATH ('COUNTRY_NAME'::text) NOT NULL, country_id text PATH ('COUNTRY_ID'::text), region_id integer PATH ('REGION_ID'::text), size double precision PATH ('SIZE'::text), unit text PATH ('SIZE/@unit'::text), premier_name text DEFAULT ('not specified'::text) PATH ('PREMIER_NAME'::text)) Table Function Call: XMLTABLE(('/ROWS/ROW'::text) PASSING (xmldata.data) COLUMNS id integer PATH ('@id'::text), _id FOR ORDINALITY, country_name text PATH ('COUNTRY_NAME/text()'::text) NOT NULL, country_id text PATH ('COUNTRY_ID'::text), region_id integer PATH ('REGION_ID'::text), size double precision PATH ('SIZE'::text), unit text PATH ('SIZE/@unit'::text), premier_name text DEFAULT ('not specified'::text) PATH ('PREMIER_NAME'::text))
(7 rows) (7 rows)
-- XMLNAMESPACES tests -- XMLNAMESPACES tests
......
src/test/regress/expected/xml_2.out
View file @ b7f0be9a
...@@ -1004,7 +1004,7 @@ SELECT xmltable.* ...@@ -1004,7 +1004,7 @@ SELECT xmltable.*
PASSING data PASSING data
COLUMNS id int PATH '@id', COLUMNS id int PATH '@id',
_id FOR ORDINALITY, _id FOR ORDINALITY,
country_name text PATH 'COUNTRY_NAME' NOT NULL, country_name text PATH 'COUNTRY_NAME/text()' NOT NULL,
country_id text PATH 'COUNTRY_ID', country_id text PATH 'COUNTRY_ID',
region_id int PATH 'REGION_ID', region_id int PATH 'REGION_ID',
size float PATH 'SIZE', size float PATH 'SIZE',
...@@ -1026,7 +1026,7 @@ CREATE VIEW xmltableview1 AS SELECT xmltable.* ...@@ -1026,7 +1026,7 @@ CREATE VIEW xmltableview1 AS SELECT xmltable.*
PASSING data PASSING data
COLUMNS id int PATH '@id', COLUMNS id int PATH '@id',
_id FOR ORDINALITY, _id FOR ORDINALITY,
country_name text PATH 'COUNTRY_NAME' NOT NULL, country_name text PATH 'COUNTRY_NAME/text()' NOT NULL,
country_id text PATH 'COUNTRY_ID', country_id text PATH 'COUNTRY_ID',
region_id int PATH 'REGION_ID', region_id int PATH 'REGION_ID',
size float PATH 'SIZE', size float PATH 'SIZE',
...@@ -1055,7 +1055,7 @@ CREATE OR REPLACE VIEW public.xmltableview1 AS ...@@ -1055,7 +1055,7 @@ CREATE OR REPLACE VIEW public.xmltableview1 AS
"xmltable".premier_name "xmltable".premier_name
FROM ( SELECT xmldata.data FROM ( SELECT xmldata.data
FROM xmldata) x, FROM xmldata) x,
LATERAL XMLTABLE(('/ROWS/ROW'::text) PASSING (x.data) COLUMNS id integer PATH ('@id'::text), _id FOR ORDINALITY, country_name text PATH ('COUNTRY_NAME'::text) NOT NULL, country_id text PATH ('COUNTRY_ID'::text), region_id integer PATH ('REGION_ID'::text), size double precision PATH ('SIZE'::text), unit text PATH ('SIZE/@unit'::text), premier_name text DEFAULT ('not specified'::text) PATH ('PREMIER_NAME'::text)) LATERAL XMLTABLE(('/ROWS/ROW'::text) PASSING (x.data) COLUMNS id integer PATH ('@id'::text), _id FOR ORDINALITY, country_name text PATH ('COUNTRY_NAME/text()'::text) NOT NULL, country_id text PATH ('COUNTRY_ID'::text), region_id integer PATH ('REGION_ID'::text), size double precision PATH ('SIZE'::text), unit text PATH ('SIZE/@unit'::text), premier_name text DEFAULT ('not specified'::text) PATH ('PREMIER_NAME'::text))
EXPLAIN (COSTS OFF) SELECT * FROM xmltableview1; EXPLAIN (COSTS OFF) SELECT * FROM xmltableview1;
QUERY PLAN QUERY PLAN
----------------------------------------- -----------------------------------------
...@@ -1065,15 +1065,15 @@ EXPLAIN (COSTS OFF) SELECT * FROM xmltableview1; ...@@ -1065,15 +1065,15 @@ EXPLAIN (COSTS OFF) SELECT * FROM xmltableview1;
(3 rows) (3 rows)
EXPLAIN (COSTS OFF, VERBOSE) SELECT * FROM xmltableview1; EXPLAIN (COSTS OFF, VERBOSE) SELECT * FROM xmltableview1;
QUERY PLAN QUERY PLAN
----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- ------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
Nested Loop Nested Loop
Output: "xmltable".id, "xmltable"._id, "xmltable".country_name, "xmltable".country_id, "xmltable".region_id, "xmltable".size, "xmltable".unit, "xmltable".premier_name Output: "xmltable".id, "xmltable"._id, "xmltable".country_name, "xmltable".country_id, "xmltable".region_id, "xmltable".size, "xmltable".unit, "xmltable".premier_name
-> Seq Scan on public.xmldata -> Seq Scan on public.xmldata
Output: xmldata.data Output: xmldata.data
-> Table Function Scan on "xmltable" -> Table Function Scan on "xmltable"
Output: "xmltable".id, "xmltable"._id, "xmltable".country_name, "xmltable".country_id, "xmltable".region_id, "xmltable".size, "xmltable".unit, "xmltable".premier_name Output: "xmltable".id, "xmltable"._id, "xmltable".country_name, "xmltable".country_id, "xmltable".region_id, "xmltable".size, "xmltable".unit, "xmltable".premier_name
Table Function Call: XMLTABLE(('/ROWS/ROW'::text) PASSING (xmldata.data) COLUMNS id integer PATH ('@id'::text), _id FOR ORDINALITY, country_name text PATH ('COUNTRY_NAME'::text) NOT NULL, country_id text PATH ('COUNTRY_ID'::text), region_id integer PATH ('REGION_ID'::text), size double precision PATH ('SIZE'::text), unit text PATH ('SIZE/@unit'::text), premier_name text DEFAULT ('not specified'::text) PATH ('PREMIER_NAME'::text)) Table Function Call: XMLTABLE(('/ROWS/ROW'::text) PASSING (xmldata.data) COLUMNS id integer PATH ('@id'::text), _id FOR ORDINALITY, country_name text PATH ('COUNTRY_NAME/text()'::text) NOT NULL, country_id text PATH ('COUNTRY_ID'::text), region_id integer PATH ('REGION_ID'::text), size double precision PATH ('SIZE'::text), unit text PATH ('SIZE/@unit'::text), premier_name text DEFAULT ('not specified'::text) PATH ('PREMIER_NAME'::text))
(7 rows) (7 rows)
-- XMLNAMESPACES tests -- XMLNAMESPACES tests
......
src/test/regress/sql/xml.sql
View file @ b7f0be9a
...@@ -349,7 +349,7 @@ SELECT xmltable.* ...@@ -349,7 +349,7 @@ SELECT xmltable.*
PASSING data PASSING data
COLUMNS id int PATH '@id', COLUMNS id int PATH '@id',
_id FOR ORDINALITY, _id FOR ORDINALITY,
country_name text PATH 'COUNTRY_NAME' NOT NULL, country_name text PATH 'COUNTRY_NAME/text()' NOT NULL,
country_id text PATH 'COUNTRY_ID', country_id text PATH 'COUNTRY_ID',
region_id int PATH 'REGION_ID', region_id int PATH 'REGION_ID',
size float PATH 'SIZE', size float PATH 'SIZE',
...@@ -362,7 +362,7 @@ CREATE VIEW xmltableview1 AS SELECT xmltable.* ...@@ -362,7 +362,7 @@ CREATE VIEW xmltableview1 AS SELECT xmltable.*
PASSING data PASSING data
COLUMNS id int PATH '@id', COLUMNS id int PATH '@id',
_id FOR ORDINALITY, _id FOR ORDINALITY,
country_name text PATH 'COUNTRY_NAME' NOT NULL, country_name text PATH 'COUNTRY_NAME/text()' NOT NULL,
country_id text PATH 'COUNTRY_ID', country_id text PATH 'COUNTRY_ID',
region_id int PATH 'REGION_ID', region_id int PATH 'REGION_ID',
size float PATH 'SIZE', size float PATH 'SIZE',
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment