Re: [HACKERS] [PATCH] Generic type subscripting

2021-01-09 Thread Dian M Fay
On Sat Jan 9, 2021 at 3:34 PM EST, Pavel Stehule wrote: > so 9. 1. 2021 v 21:06 odesílatel Dian M Fay > napsal: > > > On Thu Jan 7, 2021 at 3:24 AM EST, Pavel Stehule wrote: > > > čt 7. 1. 2021 v 9:15 odesílatel Dmitry Dolgov <9erthali...@gmail.com> > > >

Re: [HACKERS] [PATCH] Generic type subscripting

2021-01-09 Thread Dian M Fay
On Thu Jan 7, 2021 at 3:24 AM EST, Pavel Stehule wrote: > čt 7. 1. 2021 v 9:15 odesílatel Dmitry Dolgov <9erthali...@gmail.com> > napsal: > > > > On Wed, Jan 06, 2021 at 09:22:53PM +0100, Pavel Stehule wrote: > > > > > > this case should to raise exception - the value should be changed or > >

Re: [HACKERS] [PATCH] Generic type subscripting

2021-01-20 Thread Dian M Fay
t; > Thanks, I need to remember to not skipp doc building for testing process > even for such small changes. Hope now I didn't forget anything. > > > On Wed, Jan 20, 2021 at 09:58:43AM -0500, Dian M Fay wrote: > > > Here's a full editing pass on the documentation, with v45

Re: [HACKERS] [PATCH] Generic type subscripting

2021-01-20 Thread Dian M Fay
On Wed Jan 20, 2021 at 2:08 PM EST, Dmitry Dolgov wrote: > > On Wed, Jan 20, 2021 at 11:34:16AM -0500, Dian M Fay wrote: > > > Thanks, I need to remember to not skipp doc building for testing process > > > even for such small changes. Hope now I didn't forget anything. >

Re: [HACKERS] [PATCH] Generic type subscripting

2021-01-14 Thread Dian M Fay
On Thu Jan 14, 2021 at 10:04 AM EST, Dmitry Dolgov wrote: > > On Tue, Jan 12, 2021 at 08:02:59PM +0100, Pavel Stehule wrote: > > ne 10. 1. 2021 v 19:52 odesílatel Pavel Stehule > > napsal: > > > > I tested behaviour and I didn't find anything other than the mentioned > > issue. > > > > Now I can

Re: [HACKERS] [PATCH] Generic type subscripting

2021-01-20 Thread Dian M Fay
rrected a typo in one of the added hints. From 086a34ca860e8513484d829db1cb3f0c17c4ec1e Mon Sep 17 00:00:00 2001 From: Dian M Fay Date: Tue, 19 Jan 2021 23:44:23 -0500 Subject: [PATCH] Revise jsonb subscripting documentation --- doc/src/sgml/json.sgml | 101 +---

Re: [PATCH] postgres-fdw: column option to override foreign types

2021-03-02 Thread Dian M Fay
On Tue Mar 2, 2021 at 6:50 AM EST, Ashutosh Bapat wrote: > On Mon, Mar 1, 2021 at 12:59 PM Dian M Fay wrote: > > > > Full use of a custom data type with postgres_fdw currently requires the > > type be maintained in both the local and remote databases. `CREATE > > FO

Re: TRIM_ARRAY

2021-03-01 Thread Dian M Fay
On Mon Mar 1, 2021 at 6:53 PM EST, Vik Fearing wrote: > > This basically does what it says, and the code looks good. The > > documentation is out of alphabetical order (trim_array should appear > > under cardinality rather than over)) but good otherwise. > > Hmm. It appears between cardinality and

[PATCH] postgres-fdw: column option to override foreign types

2021-02-28 Thread Dian M Fay
, with a new regression test and documentation. From 09961e10daf72e2c1fbbdddb05b2940b2da14df0 Mon Sep 17 00:00:00 2001 From: Dian M Fay Date: Mon, 1 Mar 2021 00:44:15 -0500 Subject: [PATCH] postgres_fdw: column option to override foreign types Enabling the use_local_type option on a foreign table column

Re: [PATCH] postgres-fdw: column option to override foreign types

2021-03-04 Thread Dian M Fay
On Thu Mar 4, 2021 at 9:28 AM EST, Georgios Kokolatos wrote: > I am afraid I will have to :-1: this patch. Of course it is possible > that I am wrong, > so please correct me if you, or any other reviewers, think so. > > The problem that is intended to be solved, upon closer inspection > seems > to

Re: [PATCH] postgres-fdw: column option to override foreign types

2021-03-07 Thread Dian M Fay
; enum column yields "operator does not exist"). I'm rather wary of actually trying to depend on that behavior, although it seems probably-safe in the same character set and collation. From f7024ae2e95bd28c47267d97d7350f43a2b6c072 Mon Sep 17 00:00:00 2001 From: Dian M Fay Date: Sun, 7 Mar 2

Re: [PATCH] postgres_fdw: suppress explicit casts in text:text comparisons (was: column option to override foreign types)

2021-08-04 Thread Dian M Fay
On Sun Mar 7, 2021 at 2:37 AM EST, Dian M Fay wrote: > > What'd be better, if we could do it, is to ship the clause in > > the form > > WHERE foreigncol = 'one' > > that is, instead of plastering a cast on the Var, try to not put > > any explicit cast on the constant

Re: [PATCH] postgres_fdw: suppress explicit casts in text:text comparisons (was: column option to override foreign types)

2021-10-23 Thread Dian M Fay
On Sun Sep 5, 2021 at 6:43 PM EDT, Tom Lane wrote: > "Dian M Fay" writes: > > [ 0001-Suppress-explicit-casts-of-text-constants-in-postgre.patch ] > > I took a quick look at this. The restriction to type text seems like > very obviously a hack rather than somethin

Re: [PATCH] postgres_fdw: suppress explicit casts in text:text comparisons (was: column option to override foreign types)

2021-11-11 Thread Dian M Fay
On Thu Nov 11, 2021 at 3:36 PM EST, Tom Lane wrote: > I thought about this some more and realized exactly why I wanted to > restrict the change to cases where the other side is a plain foreign > Var: that way, if anything surprising happens, we can blame it > directly on the user having declared a

Re: [PATCH] postgres_fdw: suppress explicit casts in text:text comparisons (was: column option to override foreign types)

2021-11-10 Thread Dian M Fay
51086cf1b19647102679149214565 Mon Sep 17 00:00:00 2001 From: Dian M Fay Date: Mon, 8 Nov 2021 22:52:07 -0500 Subject: [PATCH v7 1/2] Suppress explicit casts of safe Consts in postgres_fdw Comparisons between Consts of UNKNOWN type (string literals or NULLs) and Vars can rely on the remote serve

Re: [PATCH] postgres_fdw: suppress explicit casts in text:text comparisons (was: column option to override foreign types)

2021-11-07 Thread Dian M Fay
. I allowed RelabelTypes over Vars to suppress casts as well. It's working for me so far and the varchar comparison tests are back to passing, sans casts. From e0fe4e4b15403ffab2518f945c13ce0e22bb89a9 Mon Sep 17 00:00:00 2001 From: Dian M Fay Date: Sun, 7 Nov 2021 17:53:17 -0500 Subject: [PATCH v6]

Re: doc: Make selectivity example match wording

2022-07-02 Thread Dian M Fay
On Thu Jun 9, 2022 at 11:57 AM EDT, David G. Johnston wrote: > Reposting this to its own thread. > > https://www.postgresql.org/message-id/flat/CAKFQuwby1aMsJDMeibaBaohgoaZhivAo4WcqHC1%3D9-GDZ3TSng%40mail.gmail.com > > doc: make unique non-null join selectivity example match the prose > >

Re: doc: Make selectivity example match wording

2022-07-17 Thread Dian M Fay
On Sat Jul 16, 2022 at 11:23 PM EDT, David G. Johnston wrote: > Thanks for the review. I generally like everything you said but it made me > realize that I still didn't really understand the intent behind the > formula. I spent way too much time working that out for myself, then > turned what I