Re: [PATCH] contrib/xml2: xslt_process() should report XSLT-related error details - Mailing list pgsql-hackers

From Robin Haberkorn
Subject Re: [PATCH] contrib/xml2: xslt_process() should report XSLT-related error details
Date
Msg-id D9I86M48OZ07.19RYHTDE8ANDM@b1-systems.de
Whole thread Raw
In response to [PATCH] contrib/xml2: xslt_process() should report XSLT-related error details  ("Robin Haberkorn" <haberkorn@b1-systems.de>)
List pgsql-hackers
On Tue Apr 22, 2025 at 17:47:20 GMT +03, Robin Haberkorn wrote:
> Perhaps you can tell me what else is preventing adoption into core. I believe
> that xslt_process() should also accept the `xml` type as an alternative to
> strings. Strings should be kept for backwards compatibility, though. Also,
> the stylesheet parameter passing is very crude and limited. I suggest, that
> the current method of passing them as strings (see parse_params()) is
> deprecated and we don't try to tackle its limitations, but will instead
> accept hstores. If you agree, I will work on these two tasks next.

On the other hand, since hstore is a module, it seems we cannot really
have any other module or core depend on it, correct?
Perhaps the xmltype-variant of xslt_process() could accept string arrays
instead.

--
Robin Haberkorn
Senior Software Engineer

B1 Systems GmbH
Osterfeldstraße 7 / 85088 Vohburg / https://www.b1-systems.de
GF: Ralph Dehner / Unternehmenssitz: Vohburg / AG: Ingolstadt, HRB 3537



pgsql-hackers by date:

Previous
From: Nisha Moond
Date:
Subject: Re: Fix slot synchronization with two_phase decoding enabled
Next
From: Shayon Mukherjee
Date:
Subject: Re: [PATCH] Re: Proposal to Enable/Disable Index using ALTER INDEX