Re: parse_subscription_options - suggested improvements

2021-08-07 Thread Peter Smith
v11 -> v12 * A rebase was needed due to some recent pgindent changes on HEAD. -- Kind Regards, Peter Smith. Fujitsu Australia v12-0001-Simplify-recent-parse_subscription_option-change.patch Description: Binary data

Typo in subscription TAP test comment

2021-08-07 Thread Peter Smith
PSA a patch to fix trivial comment typo in newly committed TAP test. "amd" -> "and" -- Kind Regards, Peter Smith. Fujitsu Australia v1-0001-Fix-typo-in-TAP-test-comment.patch Description: Binary data

Re: row filtering for logical replication

2021-08-07 Thread Peter Smith
v22 --> v23 Changes: * A rebase was needed (due to commit [1]) to keep the patch working with cfbot. -- [1] https://github.com/postgres/postgres/commit/93d573d86571d148e2d14415166ec6981d34ea9d Kind Regards, Peter Smith. Fujitsu Australia v23-0001-Row-filter-for-logical-replication.patch

Re: elog.c query_id support vs shutdown

2021-08-07 Thread Julien Rouhaud
On Sun, Aug 08, 2021 at 01:46:39PM +0800, Julien Rouhaud wrote: > On Sat, Aug 07, 2021 at 04:44:07PM -0700, Andres Freund wrote: > > > > As currently implemented those pgstat_get_my_query_id() calls are not > > safe. It's fine during backend startup because MyBEEntry is not set, but > > during shu

Re: elog.c query_id support vs shutdown

2021-08-07 Thread Julien Rouhaud
On Sat, Aug 07, 2021 at 04:44:07PM -0700, Andres Freund wrote: > > As currently implemented those pgstat_get_my_query_id() calls are not > safe. It's fine during backend startup because MyBEEntry is not set, but > during shutdown that's not ok, because we never unset MyBEEntry. > > andres@awork3:

Re: Small documentation improvement for ALTER SUBSCRIPTION

2021-08-07 Thread Peter Smith
On Sat, Aug 7, 2021 at 4:33 PM Amit Kapila wrote: > > On Thu, Jul 8, 2021 at 6:31 PM Masahiko Sawada wrote: > > > > Hi all, > > > > When reading the doc of ALTER SUBSCRIPTION I realized that 'refresh > > options' in the following paragraph is not tagged: > > > > --- > > Additionally, refresh opti

Re: Assert triggered during RE_compile_and_cache

2021-08-07 Thread Tom Lane
I wrote: > ... What surprises me about this is not that > crash, but that we didn't see fundamental breakage of backref-using > patterns all over the place. It evidently breaks only in corner > cases, but I'm not quite sure why the effects are so limited. Ah-hah, I've sussed it. I'd supposed tha

elog.c query_id support vs shutdown

2021-08-07 Thread Andres Freund
Hi, In the course of [1] I again looked at all pgstat_* callsites and noticed the new calls to pgstat_get_my_query_id() in elog.c. As currently implemented those pgstat_get_my_query_id() calls are not safe. It's fine during backend startup because MyBEEntry is not set, but during shutdown that's

Re: perlcritic: prohibit map and grep in void conext

2021-08-07 Thread Dagfinn Ilmari Mannsåker
ilm...@ilmari.org (Dagfinn Ilmari Mannsåker) writes: > Hi hackers, > > In the patches for improving the MSVC build process, I noticed a use of > `map` in void context. This is considered bad form, and has a > perlcritic policy forbidding it: > https://metacpan.org/pod/Perl::Critic::Policy::Builti

Re: [PATCH] Add tab-complete for backslash commands

2021-08-07 Thread Dagfinn Ilmari Mannsåker
Hi Tang, "tanghy.f...@fujitsu.com" writes: > Hi > > Attached a patch to improve the tab completion for backslash commands. > I think it’s common for some people(I'm one of them) to use full-name > commands than abbreviation. So it's more convenient if we can add the > full-name backslash comman

Re: ECPG bug fix: DECALRE STATEMENT and DEALLOCATE, DESCRIBE

2021-08-07 Thread Peter Geoghegan
On Sat, Aug 7, 2021 at 12:43 PM Michael Meskes wrote: > Again, I didn't know the RMT was expecting anything from me. Yes, I > knew I needed to spend some time on a technical issues, but that's > exactly the information I had at the time. As Andrew mentioned, I sent you an email on the 30th -- a f

Re: [PATCH] Tab completion for ALTER TABLE … ADD …

2021-08-07 Thread Dagfinn Ilmari Mannsåker
ilm...@ilmari.org (Dagfinn Ilmari Mannsåker) writes: > Hi Hackers, > > The other day I noticed that there's no tab completion after ALTER TABLE > … ADD, so here's a patch. In addition to COLUMN and all the table > constraint types, it also completes with the list of unique indexes on > the table

Re: [PATCH] Add tab completion for EXECUTE after EXPLAIN

2021-08-07 Thread Dagfinn Ilmari Mannsåker
ilm...@ilmari.org (Dagfinn Ilmari Mannsåker) writes: > Hi hackers, > > Here's a patch adding EXECUTE to the list of of commands tab completed > after the various EXPLAIN variants. Added to the 2021-09 commitfest: https://commitfest.postgresql.org/34/3279/ - ilmari

Re: very long record lines in expanded psql output

2021-08-07 Thread Andrew Dunstan
On 8/7/21 10:56 AM, Platon Pronko wrote: > Hi! > >> I also find this annoying and would be happy to be rid of it. > > Have you tried "\pset format wrapped"? Pavel suggested it, and it > solved most of the problem for me, for example. Yes, but it changes the data line out

Re: Assert triggered during RE_compile_and_cache

2021-08-07 Thread Tom Lane
I wrote: >> Hm. I thought that this might be an old issue, but I'm not seeing the >> crash in pre-v14 branches. That means it's some bug introduced in >> the performance improvements I did a few months ago. Open item added. > git bisect says the trouble started with > ea1268f6301cc7adce571cc9c5

[PATCH] Add tab completion for EXECUTE after EXPLAIN

2021-08-07 Thread Dagfinn Ilmari Mannsåker
Hi hackers, Here's a patch adding EXECUTE to the list of of commands tab completed after the various EXPLAIN variants. - ilmari >From a303b7b2e505d4cdd65a0bd1fdc031488a14 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Dagfinn=20Ilmari=20Manns=C3=A5ker?= Date: Wed, 4 Aug 2021 12:07:24 +0100 Subjec

Re: Commitfest overflow

2021-08-07 Thread David G. Johnston
On Thu, Aug 5, 2021 at 7:36 AM Robert Haas wrote: > Patches that are not being updated regularly have no > business being part of a CommitFest. > As the main issue seems to be "Needs Review" getting punted, the patch author rightly expects feedback before supplying new patches. If they are wait

Re: Tab completion for CREATE SCHEMAAUTHORIZATION

2021-08-07 Thread Dagfinn Ilmari Mannsåker
ilm...@ilmari.org (Dagfinn Ilmari Mannsåker) writes: > ilm...@ilmari.org (Dagfinn Ilmari Mannsåker) writes: > >> Hi Hackers, >> >> I just noticed there's no tab completion for CREATE SCHEMA >> AUTHORIZATION, nor for anything after CREATE SCHEMA . >> >> Please find attached a patch that adds this.

Re: Commitfest overflow

2021-08-07 Thread David G. Johnston
On Tue, Aug 3, 2021 at 8:53 AM Simon Riggs wrote: > There are 273 patches in the queue for the Sept Commitfest already, so > it seems clear the queue is not being cleared down each CF as it was > before. We've been trying hard, but it's overflowing. > > Of those, about 50 items have been waiting

Re: ECPG bug fix: DECALRE STATEMENT and DEALLOCATE, DESCRIBE

2021-08-07 Thread Andrew Dunstan
On 8/7/21 3:43 PM, Michael Meskes wrote: > >> No other committer (certainly nobody on the RMT) knows anything about >> ecpg. How much longer were you expecting us to wait for a simple >> status update? > Where did I say I expect you to wait? How could I even do that given > that I didn't even kno

Re: Refactor ECPGconnect and allow IPv6 connection

2021-08-07 Thread David G. Johnston
On Mon, Jun 21, 2021 at 3:46 AM kuroda.hay...@fujitsu.com < kuroda.hay...@fujitsu.com> wrote: > I will try to remake patches based on the idea. > Based upon this comment, and the ongoing discussion about commitfest volume and complexity, I've moved this to "Waiting on Author". David J.

Re: Commitfest overflow

2021-08-07 Thread David G. Johnston
On Tue, Aug 3, 2021 at 8:53 AM Simon Riggs wrote: > There are 273 patches in the queue for the Sept Commitfest already, so > it seems clear the queue is not being cleared down each CF as it was > before. We've been trying hard, but it's overflowing. > I think one simple change here would be of s

Re: ECPG bug fix: DECALRE STATEMENT and DEALLOCATE, DESCRIBE

2021-08-07 Thread Michael Meskes
> That's simply not true. Andrew Dunstan reached out personally and got > no response. He then reached out through a backchannel (a direct > coworker of yours), before finally getting a single terse response > from you here. You do know that I did not receive any email from Andrew. After all I exp

Re: Commitfest overflow

2021-08-07 Thread Dmitry Dolgov
> On Tue, Aug 03, 2021 at 02:57:49PM -0400, Bruce Momjian wrote: > > On Tue, Aug 3, 2021 at 08:51:57PM +0200, Tomas Vondra wrote: > > How would this be different from the CFM just rejecting patches? It does not > > matter if there's an explicit number of patches that we allow to be moved to > > th

Re: ECPG bug fix: DECALRE STATEMENT and DEALLOCATE, DESCRIBE

2021-08-07 Thread Peter Geoghegan
On Sat, Aug 7, 2021 at 1:13 AM Michael Meskes wrote: > I get it that the goal is to release PostgreSQL 14 and I also get it > that nobody is interested in the reasons for my slow reaction. I even, > at least to an extend, understand why nobody tried reaching out to me > directly. That's simply no

Re: very long record lines in expanded psql output

2021-08-07 Thread Platon Pronko
Hi! I also find this annoying and would be happy to be rid of it. Have you tried "\pset format wrapped"? Pavel suggested it, and it solved most of the problem for me, for example. Yes, but it changes the data line output. Ideally, you should be able to  modify these independently. I agree,

RE: [BUG] wrong refresh when ALTER SUBSCRIPTION ADD/DROP PUBLICATION

2021-08-07 Thread houzj.f...@fujitsu.com
On Sat, Aug 7, 2021 1:36 PM Amit Kapila wrote: > On Fri, Aug 6, 2021 at 9:57 PM Japin Li wrote: > > > > > > > > Hmm yes, it cannot cover all cases. I had somehow misunderstood that > > > the subscriber knows which relations are associated with which > > > publications. Given that the subscriber d

Re: [BUG] wrong refresh when ALTER SUBSCRIPTION ADD/DROP PUBLICATION

2021-08-07 Thread Li Japin
> On Aug 7, 2021, at 1:35 PM, Amit Kapila wrote: > > On Fri, Aug 6, 2021 at 9:57 PM Japin Li wrote: >> >>> >>> Hmm yes, it cannot cover all cases. I had somehow misunderstood that >>> the subscriber knows which relations are associated with which >>> publications. Given that the subscriber d

Re: alter table set TABLE ACCESS METHOD

2021-08-07 Thread Michael Paquier
On Fri, Jul 30, 2021 at 02:18:02PM -0700, Jeff Davis wrote: > It sounds like anything we do here should be part of a larger change to > make it consistent. So I'm fine with the patch you posted. As a matter of curiosity, I have checked how it would look to handle the no-op case for the sub-command

Re: Gather performance analysis

2021-08-07 Thread Dilip Kumar
On Fri, Aug 6, 2021 at 2:00 PM Dilip Kumar wrote: > > Experiment #1: > As part of this experiment, I have modified the sender to keep the > local copy of "mq_bytes_read" and "mq_bytes_written" in the local mqh > handle so that we don't need to frequently read/write cache sensitive > shared memory

Re: ECPG bug fix: DECALRE STATEMENT and DEALLOCATE, DESCRIBE

2021-08-07 Thread Michael Meskes
Hi Peter, > The RMT continues to be concerned about the lack of progress on this > open item, especially the lack of communication from Michael Meskes, > the committer of the original patch (commit ad8305a). We are prepared > to exercise our authority to resolve open items directly. The only > fal

Re: Implementing Incremental View Maintenance

2021-08-07 Thread Zhihong Yu
On Sat, Aug 7, 2021 at 12:00 AM Zhihong Yu wrote: > > > On Sun, Aug 1, 2021 at 11:30 PM Yugo NAGATA wrote: > >> Hi hackers, >> >> On Mon, 19 Jul 2021 09:24:30 +0900 >> Yugo NAGATA wrote: >> >> > On Wed, 14 Jul 2021 21:22:37 +0530 >> > vignesh C wrote: >> >> > > The patch does not apply on Head