Re: Allow file inclusion in pg_hba and pg_ident files

2022-11-13 Thread Julien Rouhaud
Hi, On Mon, Nov 14, 2022 at 02:40:37PM +0900, Michael Paquier wrote: > On Sat, Nov 12, 2022 at 04:13:53PM +0800, Julien Rouhaud wrote: > > It's looks good to me. I agree that file name and line number should be > > enough > > to diagnose any unexpected error. > > Thanks for checking. I have loo

Re: ExecRTCheckPerms() and many prunable partitions

2022-11-13 Thread Amit Langote
On Sat, Nov 12, 2022 at 1:46 AM Tom Lane wrote: > Alvaro Herrera writes: > > On 2022-Oct-06, Amit Langote wrote: > >> Actually, List of Bitmapsets turned out to be something that doesn't > >> just-work with our Node infrastructure, which I found out thanks to > >> -DWRITE_READ_PARSE_PLAN_TREES.

Re: Making Bitmapsets be valid Nodes

2022-11-13 Thread Amit Langote
On Mon, Nov 14, 2022 at 12:23 AM Tom Lane wrote: > Peter Eisentraut writes: > > On 11.11.22 21:05, Tom Lane wrote: > >> Attached is a cleaned-up version of Amit's patch v24-0003 at [2]. > >> I fixed the problems with not always tagging Bitmapsets, and changed > >> the outfuncs/readfuncs logic so

Re: Non-decimal integer literals

2022-11-13 Thread John Naylor
On Mon, Oct 10, 2022 at 9:17 PM Peter Eisentraut < peter.eisentr...@enterprisedb.com> wrote: > Taking another look around ecpg to see how this interacts with C-syntax > integer literals. I'm not aware of any particular issues, but it's > understandably tricky. I can find no discussion in the arc

Re: Time delayed LR (WAS Re: logical replication restrictions)

2022-11-13 Thread Amit Kapila
On Mon, Nov 14, 2022 at 12:14 PM Amit Kapila wrote: > > On Sat, Nov 12, 2022 at 7:21 PM vignesh C wrote: > > > > Few comments: > > 1) I feel if the user has specified a long delay there is a chance > > that replication may not continue if the replication slot falls behind > > the current LSN by m

Re: Schema variables - new implementation for Postgres 15

2022-11-13 Thread Sergey Shinderuk
On 13.11.2022 20:59, Pavel Stehule wrote: fresh rebase Hello, Sorry, I haven't been following this thread, but I'd like to report a memory management bug. I couldn't apply the latest patches, so I tested with v20221104-1-* patches applied atop of commit b0284bfb1db. postgres=# create vari

Time delayed LR (WAS Re: logical replication restrictions)

2022-11-13 Thread Amit Kapila
Hi, The thread title doesn't really convey the topic under discussion, so changed it. IIRC, this has been mentioned by others as well in the thread. On Sat, Nov 12, 2022 at 7:21 PM vignesh C wrote: > > Few comments: > 1) I feel if the user has specified a long delay there is a chance > that repl

Re: Allow file inclusion in pg_hba and pg_ident files

2022-11-13 Thread Michael Paquier
On Sat, Nov 12, 2022 at 04:13:53PM +0800, Julien Rouhaud wrote: > It's looks good to me. I agree that file name and line number should be > enough > to diagnose any unexpected error. Thanks for checking. I have looked at 0001 and 0002 again with a fresh mind, and applied both of them this morni

RE: [PATCH] Support % wildcard in extension upgrade filenames

2022-11-13 Thread Regina Obe
> Re: Sandro Santilli > > I'm attaching an updated version of the patch. This time the patch is > > tested. Nothing changes unless the .control file for the subject > > extension doesn't have a "wildcard_upgrades = true" statement. > > > > When wildcard upgrades are enabled, a file with a "%" symbo

Re: Add test module for Custom WAL Resource Manager feature

2022-11-13 Thread Bharath Rupireddy
On Sat, Nov 12, 2022 at 4:40 AM Jeff Davis wrote: > > On Fri, 2022-11-11 at 17:01 +0530, Bharath Rupireddy wrote: > > Hi, > > > > Inspired by recent commits 9fcdf2c, e813e0e and many small test > > modules/extensions under src/test/modules, I would like to propose > > one > > such test module for

Re: Add palloc_aligned() to allow arbitrary power of 2 memory alignment

2022-11-13 Thread John Naylor
On Tue, Nov 8, 2022 at 8:57 AM David Rowley wrote: > On Tue, 8 Nov 2022 at 05:24, Andres Freund wrote: > > I doubtthere's ever a need to realloc such a pointer? Perhaps we could just > > elog(ERROR)? > > Are you maybe locked into just thinking about your current planned use > case that we want t

Re: Suppressing useless wakeups in walreceiver

2022-11-13 Thread Thomas Munro
On Mon, Nov 14, 2022 at 1:05 PM Nathan Bossart wrote: > Yeah, I'm able to sort-of reproduce the problem by sending fewer replies, > but it's not clear to me why that's the problem. AFAICT all of the code > paths that write/flush are careful to send a reply shortly afterward, which > should keep w

Re: Bitmapsets as Nodes

2022-11-13 Thread Amit Langote
On Mon, Oct 17, 2022 at 6:30 PM Amit Langote wrote: > For a couple of patches that I am working on ([1], [2]), I have needed > to put Bitmapsets into a List that is in turn part of a Plan tree or a > Node tree that may be written (using outNode) and read (using > nodeRead). Bitmapsets not being a

RE: logical replication restrictions

2022-11-13 Thread Takamichi Osumi (Fujitsu)
On Tuesday, November 8, 2022 2:27 PM Kuroda, Hayato/黒田 隼人 wrote: > If you could not allocate a time to discuss this problem because of other > important tasks or events, we would like to take over the thread and modify > your patch. > > We've planned that we will start to address comments and re

Re: Suppressing useless wakeups in walreceiver

2022-11-13 Thread Nathan Bossart
On Mon, Nov 14, 2022 at 12:51:14PM +1300, Thomas Munro wrote: > On Mon, Nov 14, 2022 at 12:35 PM Thomas Munro wrote: >> Maybe there is a better way to code this (I mean, who likes global >> variables?) and I need to test some more, but I suspect the attached >> is approximately what we missed. >

Re: CI and test improvements

2022-11-13 Thread Justin Pryzby
On Fri, Nov 04, 2022 at 06:59:46PM -0700, Andres Freund wrote: > Hi, > > On 2022-11-04 18:54:12 -0500, Justin Pryzby wrote: > > Subject: [PATCH 1/8] meson: PROVE is not required > > Subject: [PATCH 3/8] meson: rename 'main' tasks to 'regress' and 'isolation' > > Pushed, thanks for the patches. T

Re: Suppressing useless wakeups in walreceiver

2022-11-13 Thread Thomas Munro
On Mon, Nov 14, 2022 at 12:35 PM Thomas Munro wrote: > Maybe there is a better way to code this (I mean, who likes global > variables?) and I need to test some more, but I suspect the attached > is approximately what we missed. Erm, ENOCOFFEE, sorry that's not quite right, it needs the apply LSN,

Re: Suppressing useless wakeups in walreceiver

2022-11-13 Thread Thomas Munro
On Mon, Nov 14, 2022 at 12:11 PM Thomas Munro wrote: > On Mon, Nov 14, 2022 at 11:26 AM Nathan Bossart > wrote: > > On Sun, Nov 13, 2022 at 05:08:04PM -0500, Tom Lane wrote: > > > There is something very seriously wrong with this patch. > > > > > > On my machine, running "make -j10 check-world" (

Re: Suppressing useless wakeups in walreceiver

2022-11-13 Thread Thomas Munro
On Mon, Nov 14, 2022 at 11:26 AM Nathan Bossart wrote: > On Sun, Nov 13, 2022 at 05:08:04PM -0500, Tom Lane wrote: > > There is something very seriously wrong with this patch. > > > > On my machine, running "make -j10 check-world" (with compilation > > already done) has been taking right about 2 m

Re: Reducing power consumption on idle servers

2022-11-13 Thread Simon Riggs
On Sun, 13 Nov 2022 at 21:28, Thomas Munro wrote: > > On Mon, Nov 14, 2022 at 5:52 AM Simon Riggs > wrote: > > The attached patch is a reduced version of the original. It covers only: > > * deprecation of the promote_trigger_file - there are no tests that > > use that, hence why there is no test

Re: Suppressing useless wakeups in walreceiver

2022-11-13 Thread Nathan Bossart
On Sun, Nov 13, 2022 at 05:08:04PM -0500, Tom Lane wrote: > There is something very seriously wrong with this patch. > > On my machine, running "make -j10 check-world" (with compilation > already done) has been taking right about 2 minutes for some time. > Since this patch, it's taking around 2:45

Re: Suppressing useless wakeups in walreceiver

2022-11-13 Thread Thomas Munro
On Mon, Nov 14, 2022 at 11:08 AM Tom Lane wrote: > There is something very seriously wrong with this patch. > > On my machine, running "make -j10 check-world" (with compilation > already done) has been taking right about 2 minutes for some time. > Since this patch, it's taking around 2:45 --- I di

Re: Suppressing useless wakeups in walreceiver

2022-11-13 Thread Tom Lane
Thomas Munro writes: > And with that change and a pgindent, pushed. There is something very seriously wrong with this patch. On my machine, running "make -j10 check-world" (with compilation already done) has been taking right about 2 minutes for some time. Since this patch, it's taking around 2:

Re: Reducing power consumption on idle servers

2022-11-13 Thread Thomas Munro
On Mon, Nov 14, 2022 at 5:52 AM Simon Riggs wrote: > The attached patch is a reduced version of the original. It covers only: > * deprecation of the promote_trigger_file - there are no tests that > use that, hence why there is no test coverage for the patch > * changing the sleep time of the start

Re: proposal: possibility to read dumped table's name from file

2022-11-13 Thread Pavel Stehule
ne 13. 11. 2022 v 9:58 odesílatel Julien Rouhaud napsal: > On Sat, Nov 12, 2022 at 09:35:59PM +0100, Pavel Stehule wrote: > > Thanks for the updated patch. Apart from the function comment it looks > good to > me. > > Justin, did you have any other comment on the patch? > > > > I don't fully unde

Re: Data is copied twice when specifying both child and parent table in publication

2022-11-13 Thread vignesh C
On Fri, 11 Nov 2022 at 11:13, wangw.f...@fujitsu.com wrote: > > On Fri, Oct 21, 2022 at 17:02 PM Peter Smith wrote: > > > > Thanks for your comments. Sorry for not replying in time. > > > On Mon, Oct 17, 2022 at 4:49 PM wangw.f...@fujitsu.com > > wrote: > > > > > > On Wed, Oct 5, 2022 at 11:08

Re: Reducing power consumption on idle servers

2022-11-13 Thread Simon Riggs
On Thu, 24 Mar 2022 at 16:02, Simon Riggs wrote: > > On Thu, 24 Mar 2022 at 15:39, Robert Haas wrote: > > > > On Thu, Mar 24, 2022 at 6:59 AM Simon Riggs > > wrote: > > > The proposals of this patch are the following, each of which can be > > > independently accepted/rejected: > > > 1. fix the s

Re: Making Bitmapsets be valid Nodes

2022-11-13 Thread Tom Lane
Peter Eisentraut writes: > On 11.11.22 21:05, Tom Lane wrote: >> Attached is a cleaned-up version of Amit's patch v24-0003 at [2]. >> I fixed the problems with not always tagging Bitmapsets, and changed >> the outfuncs/readfuncs logic so that Bitmapsets still print exactly >> as they did before (t

Re: Typo about subxip in comments

2022-11-13 Thread Japin Li
On Sat, 12 Nov 2022 at 12:12, Amit Kapila wrote: > On Fri, Nov 11, 2022 at 2:45 PM Japin Li wrote: >> Maybe a wrong plural in XidInMvccSnapshot(). >> >> * Make a quick range check to eliminate most XIDs without looking at the >> * xip arrays. >> >> I think we should use "xip array" in

Re: Schema variables - new implementation for Postgres 15

2022-11-13 Thread Pavel Stehule
pá 4. 11. 2022 v 15:28 odesílatel Dmitry Dolgov <9erthali...@gmail.com> napsal: > > On Fri, Nov 04, 2022 at 03:17:18PM +0100, Pavel Stehule wrote: > > > I've stumbled upon something that looks weird to me (inspired by the > > > example from tests): > > > > > > =# create variable v2 as int; > >

Re: Making Bitmapsets be valid Nodes

2022-11-13 Thread Peter Eisentraut
On 11.11.22 21:05, Tom Lane wrote: Per the discussion at [1], it seems like it'd be a good idea to make Bitmapsets into full-fledged, tagged Nodes, so that we could do things like print or copy lists of them without special-case logic. The extra space for the NodeTag is basically free due to ali

Re: New docs chapter on Transaction Management and related changes

2022-11-13 Thread Laurenz Albe
On Thu, 2022-11-10 at 12:17 +0100, Alvaro Herrera wrote: > On 2022-Nov-10, Laurenz Albe wrote: > > On Wed, 2022-11-09 at 09:16 -0500, Robert Treat wrote: > > > > > -  If AND CHAIN is specified, a new > > > > > transaction is > > > > > +  If AND CHAIN is specified, a new unaborted > > > >

Re: libpq error message refactoring

2022-11-13 Thread Peter Eisentraut
On 09.11.22 13:29, Alvaro Herrera wrote: +/* + * Append a formatted string to the given buffer, after translation. A + * newline is automatically appended; the format should not end with a + * newline. + */ I find the "after translation" bit unclear -- does it mean that the caller should have

Re: refactor ownercheck and aclcheck functions

2022-11-13 Thread Peter Eisentraut
On 09.11.22 19:12, Corey Huinker wrote: After considering this again, I decided to brute-force this and get rid of all the trivial wrapper functions and also several of the special cases.  That way, there is less confusion at the call sites about why this or that style is used in

Re: libpq support for NegotiateProtocolVersion

2022-11-13 Thread Peter Eisentraut
On 11.11.22 23:28, Jacob Champion wrote: Consider the case where the server sends a NegotiateProtocolVersion with a reasonable length, but then runs over its own message (either by sending an unterminated string as one of the extension names, or by sending a huge extension number). When I test th

Re: proposal: possibility to read dumped table's name from file

2022-11-13 Thread Julien Rouhaud
On Sat, Nov 12, 2022 at 09:35:59PM +0100, Pavel Stehule wrote: Thanks for the updated patch. Apart from the function comment it looks good to me. Justin, did you have any other comment on the patch? > > I don't fully understand the part about subpatterns, but is that necessary > > to > > descri