public inbox for [email protected]  

help / color / mirror / Atom feed

From: Peter Eisentraut <[email protected]>
To: David E. Wheeler <[email protected]>
To: Tom Lane <[email protected]>
Cc: jian he <[email protected]>
Cc: PostgreSQL Hackers <[email protected]>
Subject: Re: Document DateStyle effect on jsonpath string()
Date: Tue, 10 Sep 2024 22:10:47 +0200
Message-ID: <[email protected]> (raw)
In-Reply-To: <[email protected]>
References: <[email protected]>
	<CACJufxHREyTFzKOWz3f-yD14HxAdfeP1vGQQ2SQrusZ+ziRJyA@mail.gmail.com>
	<[email protected]>
	<CACJufxHE6q47-GEf3Kk0pD+Snr3yZMM4Byig2=bQKV8js7sP-A@mail.gmail.com>
	<[email protected]>
	<[email protected]>
	<[email protected]>
	<[email protected]>
	<[email protected]>

Isn't this behavior actually a bug that should be fixed rather than 
documented?

These JSON path functions are specified by the SQL standard, so they 
shouldn't depend on PostgreSQL-specific settings.  At least in new 
functionality we should avoid that, no?


On 10.09.24 21:43, David E. Wheeler wrote:
> On Sep 10, 2024, at 14:51, Tom Lane <[email protected]> wrote:
> 
>> Pushed with a little additional polishing.
> 
> Thank you! Do you think it’d be worthwhile to back port to 17?
> 
>> I thought the best way to address jian's complaint about DateStyle not
>> being clearly locked down was to change horology.sql to verify the
>> prevailing setting, as it has long done for TimeZone.  That's the
>> lead test script for related stuff, so it makes the most sense to
>> do it there.  Having done that, I don't feel a need to duplicate
>> that elsewhere.
> 
> Yeah, that will help, but I still bet next time I go to figure out what it is I’ll stick that line in some test to make it fail with clear output for what it’s set to 😂.

view thread (14+ messages)  latest in thread

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: [email protected]
  Cc: [email protected], [email protected], [email protected], [email protected], [email protected]
  Subject: Re: Document DateStyle effect on jsonpath string()
  In-Reply-To: <[email protected]>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

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