> On 17 Nov 2022, at 08:06, Japin Li wrote:
> I find some typo about xl_running_xacts in comments.
> Attached a patch to fix those.
Thanks, applied!
> - * might look that we could use xl_running_xact's ->xids information to
> + * might look that we could use xl_running_xacts->xids inf
On Thu, 17 Nov 2022 at 16:22, Daniel Gustafsson wrote:
>> On 17 Nov 2022, at 08:06, Japin Li wrote:
>
>> I find some typo about xl_running_xacts in comments.
>> Attached a patch to fix those.
>
> Thanks, applied!
>
>> - * might look that we could use xl_running_xact's ->xids information to
On Thu, Nov 17, 2022 at 4:46 AM Tom Lane wrote:
> So we've marked the 4 and 7 joins as possibly commuting, but they
> cannot commute according to 7's min_lefthand set. I don't think
> the extra clone condition is terribly harmful --- it's useless
> but shouldn't cause any problems. However, if
> On 17 Nov 2022, at 02:31, Richard Guo wrote:
> On Wed, Nov 16, 2022 at 4:37 PM Waithant Myo (Fujitsu)
> mailto:myo.waith...@fujitsu.com>> wrote:
> Therefore, I attached the patch file which fix this word usage.
>
> It would be great if you could take a look at it.
>
> Apparently this is a ty
Hi,
On Wed, Nov 16, 2022 at 11:26:09PM -0800, Maciek Sakrejda wrote:
> I noticed an odd behavior today in pg_stat_statements query
> normalization for queries called from SQL-language functions. If I
> have three functions that call an essentially identical query (the
> functions are only marked S
On Thu, Nov 17, 2022 at 2:44 PM Julien Rouhaud wrote:
>
> Hi,
>
> On Wed, Nov 16, 2022 at 11:26:09PM -0800, Maciek Sakrejda wrote:
> > I noticed an odd behavior today in pg_stat_statements query
> > normalization for queries called from SQL-language functions. If I
> > have three functions that ca
On 2022-Nov-16, Thom Brown wrote:
> Once the issue Tom identified has been resolved, I'd like to test
> drive newer patches.
What issue? If you mean the one from the thread "Reducing
duplicativeness of EquivalenceClass-derived clauses", that patch is
already applied (commit a5fc46414deb), and Yu
Hi,
When I was reviewing one of the patches, I found a typo in the
comments of SH_LOOKUP and SH_LOOKUP_HASH. I felt "lookup up" should
have been "lookup".
Attached a patch to modify it.
Regards,
Vignesh
From 03feb3121dbd9a78547a15cf5998e44ffb65626e Mon Sep 17 00:00:00 2001
From: Vignesh C
Date:
On Wed, Nov 16, 2022 at 05:25:07PM -0600, Justin Pryzby wrote:
> Note that the patch is passing tests when using autoconf build but
> failing for meson builds.
Thanks for pointing me to the right direction.
I'm attaching an updated patch, will keep an eye on cirrusCI
--strk;
>From 344f2b96c172ed8
> On 10 Nov 2022, at 11:44, Nikolay Shaplov wrote:
> I've checked new output, if is conform TAP specification. Checked that prove
> consumes new pg_regress output well.
Great!
> Did not found quick way to include prove TAP harness right into Makefile, so
> I
> check dumped output, but it is
Hi hackers,
> 8272749e added a few more arguments to CastCreate(). Here is the rebased
> patch.
After merging afbfc029 [1] the patch needed a rebase. PFA v10.
The patch is still in a PoC state and this is exactly why comments and
suggestions from the community are most welcome! Particularly I w
On Thu, 17 Nov 2022 at 09:31, Alvaro Herrera wrote:
>
> On 2022-Nov-16, Thom Brown wrote:
>
> > Once the issue Tom identified has been resolved, I'd like to test
> > drive newer patches.
>
> What issue? If you mean the one from the thread "Reducing
> duplicativeness of EquivalenceClass-derived cl
On 16.11.22 20:25, Tom Lane wrote:
Christoph Moench-Tegeder writes:
## Peter Eisentraut (peter.eisentr...@enterprisedb.com):
First of all, is this a standard installation of this OS, or is perhaps
something incomplete, broken, or unusual about the current OS installation?
Alpine uses musl l
On 11/17/22 03:43, Andres Freund wrote:
> Hi,
>
>
> On 2022-11-17 02:41:14 +0100, Tomas Vondra wrote:
>> Well, yeah - we can either try to perform the stuff independently of the
>> transactions that triggered it, or we can try making it part of some of
>> the transactions. Each of those option
On Wed, 16 Nov 2022 at 20:41, Andres Freund wrote:
> Hi,
> You can't do catalog access below the bufmgr.c layer. It could lead to all
> kinds of nastiness, including potentially recursing back to md.c. Even
> leaving
>
Yep, this is my biggest concern. It turns out, that the way to make such a
fea
On Thu, 17 Nov 2022 at 11:20, Thom Brown wrote:
>
> On Thu, 17 Nov 2022 at 09:31, Alvaro Herrera wrote:
> >
> > On 2022-Nov-16, Thom Brown wrote:
> >
> > > Once the issue Tom identified has been resolved, I'd like to test
> > > drive newer patches.
> >
> > What issue? If you mean the one from th
Hi, hackers!
I've noticed that alignment requirements for using
pg_atomic_init_u64() for LWlock state (there's an assertion that it's
aligned at 8 bytes) do not correspond to the code in SimpleLruInit()
on 32-bit arch when MAXIMUM_ALIGNOF = 4.
Fixed this in v4 patch (PFA).
Regards,
Pavel.
v4-00
Hi,
I was looking at commit aca992040951c7665f1701cd25d48808eda7a809
I think the check of msg after the switch statement is not necessary. The
variable msg is used afterward.
If there is (potential) missing case in switch statement, the compiler
would warn.
How about removing the check ?
Thanks
> On 17 Nov 2022, at 10:56, vignesh C wrote:
> When I was reviewing one of the patches, I found a typo in the
> comments of SH_LOOKUP and SH_LOOKUP_HASH. I felt "lookup up" should
> have been "lookup".
> Attached a patch to modify it.
I agree with that, applied to HEAD. Thanks!
--
Daniel Gustaf
Hi Hackers,
while testing the developer settings of PSQL (14.5) I came across this
issue:
postgres=# CREATE UNLOGGED TABLE stats (
postgres(# pg_hash BIGINT NOT NULL,
postgres(# category TEXT NOT NULL,
postgres(# PRIMARY KEY (pg_hash, category)
postgres(# );
server closed the connecti
> On 8 Aug 2022, at 07:37, Tom Lane wrote:
> Dong Wook Lee writes:
>> I wrote a test of the old_snapshot extension for coverage.
>
> Hmm, does this really provide any meaningful coverage? The test
> sure looks like it's not doing much.
Looking at this I agree, this test doesn't provide enough
Hi,
pg_stat_bgwriter view currently reports checkpointer stats as well. It
is that way because historically checkpointer was part of bgwriter
until the commits 806a2ae and bf405ba, that went into PG 9.2,
separated them out. I think it is time for us to separate checkpointer
stats to its own view.
Hi,
Recently read the code, I find that when calling DefineIndex
from ProcessUtilitySlow, is_alter_table will be set true if
this statement is came from expandTableLikeClause.
I check the code of DefineIndex, there are only two places use
is_alter_table:
1. the function index_check_
On Thu, Nov 17, 2022 at 12:49 AM Robert Haas wrote:
>
> > I also think that a new pg_stat_checkpoint view is needed
> > because, right now, the PgStat_CheckpointerStats stats are exposed via
> > the pg_stat_bgwriter view, having a separate view for checkpoint stats
> > is good here.
>
> Yep.
On W
On Wed, Nov 16, 2022 at 10:57 PM Himanshu Upadhyaya
wrote:
> I think if pred_xmin is aborted and curr_xmin is in progress we should
> consider it as a corruption case but vice versa is not true.
Yeah, you're right. I'm being stupid about subtransactions again.
--
Robert Haas
EDB: http://www.en
On 2022-11-16 We 14:25, Tom Lane wrote:
> Christoph Moench-Tegeder writes:
>> ## Peter Eisentraut (peter.eisentr...@enterprisedb.com):
>>> First of all, is this a standard installation of this OS, or is perhaps
>>> something incomplete, broken, or unusual about the current OS installation?
>> A
On 2022-11-17 Th 01:18, Noah Misch wrote:
> On Mon, Aug 22, 2022 at 09:44:42AM -0400, Andrew Dunstan wrote:
>> On 2022-08-21 Su 20:40, Noah Misch wrote:
>>> This (commit 13d856e of 2015-07-29) added the following:
>>>
>>> --- a/src/test/perl/TestLib.pm
>>> +++ b/src/test/perl/TestLib.pm
>>> @@ -2
On Wed, Nov 16, 2022 at 2:52 PM Andres Freund wrote:
> I don't think we'd want every buffer write or whatnot go through the
> changecount mechanism, on some non-x86 platforms that could be noticable. But
> if we didn't stage the stats updates locally I think we could make most of the
> stats chang
Andrew Dunstan writes:
> malleefowl is a docker instance (mostly docker images is what Alpine is
> used for). It would be extremely easy to recreate the image and add in
> musl-locales, but maybe we should just leave it as it is to test the
> case where the command isn't available.
Agreed, leave
Hi,
I did some simple benchmark with v2 and v3, using the attached script,
which essentially just builds hash index on random data, with different
data types and maintenance_work_mem values. And what I see is this
(median of 10 runs):
machine data type m_w_mmasterv2
On 2022-11-09 We 15:25, Andres Freund wrote:
> Hi,
>
> On 2022-11-09 14:44:42 -0500, Tom Lane wrote:
>> Andres Freund writes:
>>> A second question: Both autoconf and meson print the segment size as GB
>>> right
>>> now. Obviously that'll print out a size of 0 for a segsize < 1GB.
>>> The easie
On 16.11.22 19:35, Jacob Champion wrote:
On Tue, Nov 15, 2022 at 2:19 AM Peter Eisentraut
wrote:
I think for the current code, the following would be an appropriate
adjustment:
diff --git a/src/interfaces/libpq/fe-connect.c
b/src/interfaces/libpq/fe-connect.c
index 746e9b4f1efc..d15fb96572d9 1
The attached patch is a contrib module to set login restrictions on users with
too many authentication failure. The administrator could manage several GUC
parameters to control the login restrictions which are listed below.
- set the wait time when password authentication fails.
- allow the wai
On Thu, 17 Nov 2022 at 20:12, Ted Yu wrote:
> Hi,
> I was looking at commit aca992040951c7665f1701cd25d48808eda7a809
>
> I think the check of msg after the switch statement is not necessary. The
> variable msg is used afterward.
> If there is (potential) missing case in switch statement, the com
On 17.11.22 01:27, Andrey Borodin wrote:
Also I've found one more TODO item for the patch. Currently in
fe-connect.c patch is doing buffer reset:
conn->inStart = conn->inCursor = conn->inEnd = 0;
This effectively consumes butes up tu current cursor. However, packet
inspection is continued. The pa
On Thu, 17 Nov 2022 at 23:06, Japin Li wrote:
> On Thu, 17 Nov 2022 at 20:12, Ted Yu wrote:
>> Hi,
>> I was looking at commit aca992040951c7665f1701cd25d48808eda7a809
>>
>> I think the check of msg after the switch statement is not necessary. The
>> variable msg is used afterward.
>> If there i
On Tue, Nov 08, 2022 at 07:30:38PM +0100, Sergey Dudoladov wrote:
> +Logs reception of a connection. At this point a connection
> has been received, but no further work has been done:
receipt
> +Logs the original identity that an authentication method
> employs to identify a use
Hi,
On 2022-11-17 09:58:48 -0500, Andrew Dunstan wrote:
> On 2022-11-09 We 15:25, Andres Freund wrote:
> > Hi,
> >
> > On 2022-11-09 14:44:42 -0500, Tom Lane wrote:
> >> Andres Freund writes:
> >>> A second question: Both autoconf and meson print the segment size as GB
> >>> right
> >>> now. Obv
On 2022-11-17 14:40:02 +0300, Maxim Orlov wrote:
> On Wed, 16 Nov 2022 at 20:41, Andres Freund wrote:
> > that aside, we can't do catalog accesses in all kinds of environments that
> > this currently is active in - most importantly it's affecting the startup
> > process. We don't do catalog access
Andres Freund writes:
> On 2022-11-17 09:58:48 -0500, Andrew Dunstan wrote:
>> Are we going to impose some sane minimum, or leave it up to developers
>> to discover that for themselves?
> I don't think we should. It's actually useful to e.g. use 1 page sized
> segments for testing, and with one e
Japin Li writes:
> On Thu, 17 Nov 2022 at 23:06, Japin Li wrote:
>> I think we cannot remove the check, for example, if objtype is
>> OBJECT_OPFAMILY,
>> and schema_does_not_exist_skipping() returns true, the so the msg keeps NULL,
>> if we remove this check, a sigfault might be occurd in erepor
On Wed, Nov 16, 2022 at 12:41 PM Himanshu Upadhyaya <
upadhyaya.himan...@gmail.com> wrote:
>
>
>> > + }
>> > +
>> > + /* Loop over offsets and validate the data in the
>> predecessor array. */
>> > + for (OffsetNumber currentoffnum = FirstOffsetNumber;
>> curren
Hi Jacob,
Thanks, I've added this patch in my tree [1]. (For now, just adding
fixes and the like atop the original separate patches, but will
eventually get things winnowed down into probably the same 12 parts
the originals were reviewed in.
Best,
David
[1] https://github.com/pgguru/postgres/tr
On Tue, Nov 15, 2022 at 3:32 AM Andres Freund wrote:
>
> > Furthermore, it is
> > possible that successor[x] = successor[x'] since the page might be
> corrupted
> > and we haven't checked otherwise.
> >
> > predecessor[y] = x means that successor[x] = y but in addition we've
> > checked that y is
On Wed, Nov 16, 2022 at 8:41 PM Tomas Vondra
wrote:
> There's a couple of caveats, though:
>
> 1) Maybe we should focus more on "actually observed" state instead of
> "observable". Who cares if the sequence moved forward in a transaction
> that was ultimately rolled back? No committed transaction
Spyridon Dimitrios Agathos writes:
> while testing the developer settings of PSQL (14.5) I came across this
> issue:
> postgres=# CREATE UNLOGGED TABLE stats (
> postgres(# pg_hash BIGINT NOT NULL,
> postgres(# category TEXT NOT NULL,
> postgres(# PRIMARY KEY (pg_hash, category)
> pos
Hi,
On 2022-11-17 09:03:32 -0500, Robert Haas wrote:
> On Wed, Nov 16, 2022 at 2:52 PM Andres Freund wrote:
> > I don't think we'd want every buffer write or whatnot go through the
> > changecount mechanism, on some non-x86 platforms that could be noticable.
> > But
> > if we didn't stage the st
On 2022-11-17 10:48:52 -0500, Tom Lane wrote:
> Yeah, I think we should allow setting it to 1 block. This switch is
> only for testing purposes (I hope the docs make that clear).
"This option is only for developers, to test segment related code."
On Wed, Nov 16, 2022 at 3:30 AM Bharath Rupireddy
wrote:
> 1.
> -if (config.filter_by_fpw && !XLogRecordHasFPW(xlogreader_state))
> +if (config.filter_by_fpw && !XLogRecordHasFPI(xlogreader_state))
> These changes are not related to this feature, hence renaming those
> variables/fu
Hi Dilip,
Thanks for the feedback here. I will review the docs changes and add to my tree.
Best,
David
Hi,
I have an incorrect behavior with pg_dump prior PG version 15. With
PostgreSQL 15, thanks to commit e3fcbbd623b9ccc16cdbda374654d91a4727d173
the problem is gone but for older versions it persists with locks on
partitioned tables.
When we try to dump a database where a table is locked,
Gilles Darold writes:
> I have an incorrect behavior with pg_dump prior PG version 15. With
> PostgreSQL 15, thanks to commit e3fcbbd623b9ccc16cdbda374654d91a4727d173
> the problem is gone but for older versions it persists with locks on
> partitioned tables.
I didn't want to back-patch e3fcbb
On Wed, Nov 16, 2022 at 4:34 PM Peter Geoghegan wrote:
> WFM.
Attached is v3.
Plan is to commit this later on today, barring objections.
Thanks
--
Peter Geoghegan
v3-0001-Standardize-rmgrdesc-recovery-conflict-XID-output.patch
Description: Binary data
Le 17/11/2022 à 17:59, Tom Lane a écrit :
Gilles Darold writes:
I have an incorrect behavior with pg_dump prior PG version 15. With
PostgreSQL 15, thanks to commit e3fcbbd623b9ccc16cdbda374654d91a4727d173
the problem is gone but for older versions it persists with locks on
partitioned tables.
On Wed, 16 Nov 2022 at 15:44, Tom Lane wrote:
>
> Simon Riggs writes:
> > On Wed, 16 Nov 2022 at 00:09, Tom Lane wrote:
> >> No, that's not what XidInMVCCSnapshot does. If snapshot->suboverflowed
> >> is set (ie, somebody somewhere/somewhen overflowed), then it does
> >> SubTransGetTopmostTransa
Hi,
On 2022-11-17 12:39:49 +0100, Tomas Vondra wrote:
> On 11/17/22 03:43, Andres Freund wrote:
> > On 2022-11-17 02:41:14 +0100, Tomas Vondra wrote:
> >> Well, yeah - we can either try to perform the stuff independently of the
> >> transactions that triggered it, or we can try making it part of s
Andres Freund writes:
> I think pgstat_progress_start_command() needs the changecount stuff, as does
> pgstat_progress_update_multi_param(). But for anything updating a single
> parameter at a time it really doesn't do anything useful on a platform that
> doesn't tear 64bit writes (so it could be
On Thu, Nov 17, 2022 at 11:24 AM Andres Freund wrote:
> As an experiment, I added a progress report to BufferSync()'s first loop
> (i.e. where it checks all buffers). On a 128GB shared_buffers cluster that
> increases the time for a do-nothing checkpoint from ~235ms to ~280ms. If I
> remove the ch
Hi,
On 2022-11-17 10:44:18 +0530, Amit Kapila wrote:
> On Wed, Nov 16, 2022 at 11:56 PM Andres Freund wrote:
> > On 2022-11-16 14:22:01 +0530, Amit Kapila wrote:
> > > On Wed, Nov 16, 2022 at 7:30 AM Andres Freund wrote:
> > > > On 2022-11-15 16:20:00 +0530, Amit Kapila wrote:
> > > > > On Tue,
I wrote:
> I wonder whether we ought to back-patch f10f0ae42. We could
> leave the RelationOpenSmgr macro in existence to avoid unnecessary
> breakage of extension code, but stop using it within our own code.
Concretely, about like this for v14 (didn't look at the older
branches yet).
I'm not su
Hi,
On 2022-11-17 21:33:17 +0530, Himanshu Upadhyaya wrote:
> On Tue, Nov 15, 2022 at 3:32 AM Andres Freund wrote:
> > > Furthermore, it is
> > > possible that successor[x] = successor[x'] since the page might be
> > corrupted
> > > and we haven't checked otherwise.
> > >
> > > predecessor[y] = x
Hi,
On 2022-11-17 11:36:13 +0100, Daniel Gustafsson wrote:
> > On 10 Nov 2022, at 11:44, Nikolay Shaplov wrote:
> > Did not found quick way to include prove TAP harness right into Makefile,
> > so I
> > check dumped output, but it is not really important for now, I guess.
>
> I think we'll start
On Thu, Nov 17, 2022 at 2:22 AM Bharath Rupireddy
wrote:
> Duplication is a problem that I agree with and I have an idea here -
> how about introducing a new function, say EnableStandbyMode() that
> sets StandbyMode to true and disables the startup progress timeout,
> something like the attached?
On Wed, Nov 16, 2022 at 8:42 PM Andres Freund wrote:
> Afaict the problem is that
> proc = (PGPROC *) &(waitQueue->links);
>
> is a gross gross hack - this isn't actually a PGPROC, it's pointing to an
> SHM_QUEUE, but *not* one embedded in PGPROC. It kinda works because ->links
>
On Wed, Nov 16, 2022 at 7:59 AM Maxim Orlov wrote:
> The customer, mentioned above, in this particular case, would be glad to be
> able to have a mechanism to stop the cluster.
> Again, in this concrete case.
>
> My proposal is to add a tablespace option in order to be able to configure
> which
On Wed, Nov 16, 2022 at 05:14:07PM -0500, Greg Stark wrote:
> I think this requesting autovacuum worker should be a distinct
> command. Or at least an explicit option to vacuum.
+1. I was going to suggest VACUUM (NOWAIT) ..
--
Justin
On Thu, Nov 17, 2022 at 10:56 AM Tom Lane wrote:
> This is a completely bad idea. If it takes that level of analysis
> to see that msg can't be null, we should leave the test in place.
> Any future modification of either this code or what it calls could
> break the conclusion.
+1. Also, even if
On Wed, Nov 16, 2022 at 5:14 PM Greg Stark wrote:
> However I'm not a fan of commands that sometimes do one thing and
> sometimes magically do something very different. I don't like the idea
> that the same vacuum command would sometimes run in-process and
> sometimes do this out of process reques
Hi,
On 2022-11-17 14:20:47 -0500, Robert Haas wrote:
> On Wed, Nov 16, 2022 at 8:42 PM Andres Freund wrote:
> > Afaict the problem is that
> > proc = (PGPROC *) &(waitQueue->links);
> >
> > is a gross gross hack - this isn't actually a PGPROC, it's pointing to an
> > SHM_QUEUE, bu
Andres Freund writes:
> On 2022-11-17 14:20:47 -0500, Robert Haas wrote:
>> Not that I object to a targeted fix
> Should we backpatch this fix? Likely this doesn't cause active breakage
> outside of 32bit builds under ubsan, but that's not an unreasonable thing to
> want to do in the backbranches
On 11/17/22 18:29, Simon Riggs wrote:
> On Thu, 17 Nov 2022 at 17:04, Simon Riggs
> wrote:
>>
>> New version with greatly improved comments coming very soon.
>
>>> Perhaps it would be a good idea to split up the patch. The business
>>> about making pg_subtrans flat rather than a tree seems l
On Thu, Nov 17, 2022 at 2:55 AM Simon Riggs
wrote:
> No, it will have a direct effect only on people using promote_trigger_file
> who do not read and act upon the deprecation notice before upgrading
> by making a one line change to their failover scripts.
TBH, I wonder if we shouldn't just nuke p
On 11/17/22 18:07, Andres Freund wrote:
> Hi,
>
> On 2022-11-17 12:39:49 +0100, Tomas Vondra wrote:
>> On 11/17/22 03:43, Andres Freund wrote:
>>> On 2022-11-17 02:41:14 +0100, Tomas Vondra wrote:
Well, yeah - we can either try to perform the stuff independently of the
transactions tha
On Fri, Nov 18, 2022 at 9:13 AM Andres Freund wrote:
> Agreed. I had started on a set of patches for some of the SHM_QUEUE uses, but
> somehow we ended up a bit stuck on the naming of dlist_delete variant that
> afterwards zeroes next/prev so we can replace SHMQueueIsDetached() uses.
>
> Should pr
I happened to notice that the link to Hunspell in our documentation goes to the
hunspell sourceforge page last updated in 2015. The project has since moved to
Github [0] with hunspell.sourceforge.net redirecting there, I'm not sure
exactly when but Wikipedia updated their link entry in 2016 [1] so
On Tue, Nov 15, 2022 at 7:23 PM Jeff Davis wrote:
> On Tue, 2022-11-15 at 12:07 -0500, Robert Haas wrote:
> > If anyone else wants to comment, or if either of those people want to
> > comment further, please speak up soon.
>
> Did you have some thoughts on:
>
> https://postgr.es/m/a41d606d03b6
Hi,
I wonder why the walreceiver didn't start in
008_min_recovery_point_node_3.log here:
https://buildfarm.postgresql.org/cgi-bin/show_log.pl?nm=mamba&dt=2022-11-16%2023%3A13%3A38
There was the case of commit 8acd8f86, but that involved a deadlocked
postmaster whereas this one still handled a sh
Patch: Global Unique Index
“Global unique index” in our definition is a unique index on a partitioned
table that can ensure cross-partition uniqueness using a non-partition key.
This work is inspired by this email thread, “Proposal: Global Index” started
back in 2019 (Link below). My colleag
Thomas Munro writes:
> I wonder why the walreceiver didn't start in
> 008_min_recovery_point_node_3.log here:
> https://buildfarm.postgresql.org/cgi-bin/show_log.pl?nm=mamba&dt=2022-11-16%2023%3A13%3A38
mamba has been showing intermittent failures in various replication
tests since day one. My g
On 2022-11-04 Fr 10:06, Jehan-Guillaume de Rorthais wrote:
> On Thu, 3 Nov 2022 13:11:18 -0500
> Justin Pryzby wrote:
>
>> On Tue, Jun 28, 2022 at 06:17:40PM -0400, Andrew Dunstan wrote:
>>> Nice catch, but this looks like massive overkill. I think we can very
>>> simply fix the test in just a f
On 2022-11-17 Th 10:48, Tom Lane wrote:
> Andres Freund writes:
>> On 2022-11-17 09:58:48 -0500, Andrew Dunstan wrote:
>>> Are we going to impose some sane minimum, or leave it up to developers
>>> to discover that for themselves?
>> I don't think we should. It's actually useful to e.g. use 1 pa
=?UTF-8?B?5q2j5Y2O5ZCV?= writes:
> Recently read the code, I find that when calling DefineIndex
> from ProcessUtilitySlow, is_alter_table will be set true if
> this statement is came from expandTableLikeClause.
Yeah.
> Based on the above, I think we can always a false value
> for is_a
On Fri, Nov 18, 2022 at 11:08 AM Tom Lane wrote:
> Thomas Munro writes:
> > I wonder why the walreceiver didn't start in
> > 008_min_recovery_point_node_3.log here:
> > https://buildfarm.postgresql.org/cgi-bin/show_log.pl?nm=mamba&dt=2022-11-16%2023%3A13%3A38
>
> mamba has been showing intermitte
=?UTF-8?B?5oiQ5LmL54SV?= writes:
> The attached patch is a contrib module to set login restrictions on users
> with
> too many authentication failure. The administrator could manage several GUC
> parameters to control the login restrictions which are listed below.
> - set the wait time when pa
Thomas Munro writes:
> On Fri, Nov 18, 2022 at 11:08 AM Tom Lane wrote:
>> mamba has been showing intermittent failures in various replication
>> tests since day one.
> I wonder if it's a runtime variant of the other problem. We do
> load_file("libpqwalreceiver", false) before unblocking signal
On Fri, Nov 18, 2022 at 11:35 AM Thomas Munro wrote:
> I wonder if it's a runtime variant of the other problem. We do
> load_file("libpqwalreceiver", false) before unblocking signals but
> maybe don't resolve the symbols until calling them, or something like
> that...
Hmm, no, I take that back.
On Thu, Nov 17, 2022 at 9:02 AM Peter Geoghegan wrote:
> Plan is to commit this later on today, barring objections.
Pushed, thanks.
--
Peter Geoghegan
Hi, thanks a lot for your reply!
> Why do you think this is an improvement?
I hit the issue in Greenplum Database (Massively Parallel Postgres),
the MPP architecture is that coordinator dispatch statement to segments.
The dispatch logic is quit different for AlterTable and CreateTableLike:
* alt
> On Nov 18, 2022, at 4:04 AM, Robert Haas wrote:
>
> On Thu, Nov 17, 2022 at 10:56 AM Tom Lane wrote:
>> This is a completely bad idea. If it takes that level of analysis
>> to see that msg can't be null, we should leave the test in place.
>> Any future modification of either this code or w
Hi,
On 2022-10-24 14:13:06 +0300, Melih Mutlu wrote:
> If you say so, then I think it's ready.
I pushed it with a few changes:
- I added an only_if, otherwise the task shows up (without running) even if
you use ci-os-only: linux
- I added -Dnls=disabled - It seemed to add about 1.5min to the c
On Thu, 2022-11-17 at 16:52 -0500, Robert Haas wrote:
> But I think the bigger
> reason is that, in my opinion, this proposal is more generally
> useful,
> because it takes no position on why you wish to disallow SET ROLE.
> You
> can just disallow it in some cases and allow it in others, and that
On 11/14/22 10:19, David Geier wrote:
> Hi Tom,
>> There won't *be* any MCV stats for a column that ANALYZE perceives to
>> be unique, so I'm not quite sure where the claimed savings comes from.
>
> We save if one join attribute is unique while the other isn't. In that
> case stored MCV stats ar
On Tue, Nov 15, 2022 at 8:57 PM houzj.f...@fujitsu.com
wrote:
>
> On Saturday, November 12, 2022 7:06 PM Amit Kapila
> >
> > On Fri, Nov 11, 2022 at 2:12 PM houzj.f...@fujitsu.com
> > wrote:
> > >
> >
> > Few comments on v46-0001:
> > ==
> >
>
> Thanks for the comments.
>
> >
Tomas Vondra writes:
> Or perhaps what if we have a function that quickly determines if the
> attribute has MCV, without loading it? I'd bet the expensive part of
> get_attstatslot() is the deconstruct_array().
> We could have a function that only does the first small loop over slots,
> and return
Hi,
On 2022-11-17 13:06:23 +0530, Bharath Rupireddy wrote:
> I understand. I know it's a bit hard to measure the power savings, I'm
> wondering if there's any info, maybe not necessarily related to
> postgres, but in general how much power gets saved if a certain number
> of waits/polls/system cal
On Thu, 17 Nov 2022 at 17:48, Daniel Gustafsson wrote:
>
> > On 17 Nov 2022, at 10:56, vignesh C wrote:
>
> > When I was reviewing one of the patches, I found a typo in the
> > comments of SH_LOOKUP and SH_LOOKUP_HASH. I felt "lookup up" should
> > have been "lookup".
> > Attached a patch to modi
On Thu, Nov 17, 2022 at 7:09 AM Peter Eisentraut
wrote:
>
> Note that the above code was just changed in dce92e59b1.
Thanks!
> I don't know
> how that affects this patch set.
With dce92e59b1 it would be much easier to find a bug in the compression patch.
Some more notes about the patch. (sorry f
On Tue, Oct 25, 2022 at 02:20:12PM +0900, Michael Paquier wrote:
> Attached is a rebased patch set, as of the conflicts from 2e0d80c.
So, this patch set has been sitting in the CF app for a few weeks now,
and I would like to apply them to remove a bit of code from the
executor.
Please note that i
I wrote:
> Tomas Vondra writes:
>> Or perhaps what if we have a function that quickly determines if the
>> attribute has MCV, without loading it? I'd bet the expensive part of
>> get_attstatslot() is the deconstruct_array().
>> We could have a function that only does the first small loop over slot
On Wed, Nov 16, 2022 at 1:50 PM houzj.f...@fujitsu.com
wrote:
>
> On Tuesday, November 15, 2022 7:58 PM houzj.f...@fujitsu.com
> wrote:
>
> I noticed that I didn't add CHECK_FOR_INTERRUPTS while retrying send message.
> So, attach the new version which adds that. Also attach the 0004 patch that
1 - 100 of 115 matches
Mail list logo
