babelfish-for-postgresql / babelfish_extensions / 24727542045

77%

BABEL_6_X_DEV: 77%

Build Type

push

github

Committed by web-flow

Commit Message

Fix missing xmlns:xsi namespace on elements when using FOR XML RAW('') with ELEMENTS XSINIL (#4767)

Currently, when using FOR XML RAW('') , ELEMENTS XSINIL (empty element name), Babelfish omits the xmlns:xsi namespace declaration from the output elements. T-SQL places xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" on every element when there is no row wrapper element, since there is no parent element to inherit the namespace from.

For example, SELECT 1 AS a, NULL AS b FOR XML RAW(''), ELEMENTS XSINIL produces:

Babelfish (before): <a>1</a><b xsi:nil="true"/>

T-SQL: <a xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance">1</a> <b xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:nil="true"/>

With this change, Babelfish now adds the xmlns:xsi namespace declaration to each element when RAW('') is used with ELEMENTS XSINIL, matching T-SQL behavior. This applies to both non-null elements and nil elements.

Task: BABEL-6445
Signed-off-by: Japleen Kaur <amjj@amazon.com>

Coverage Stats

6 of 6 new or added lines in 1 file covered. (100.0%)

77 existing lines in 3 files now uncovered.

53121 of 69133 relevant lines covered (76.84%)

640180.78 hits per line