Re: Is it worth accepting multiple CRLs?

2021-02-18 Thread Kyotaro Horiguchi
Thanks for committing this! At Thu, 18 Feb 2021 08:24:23 +0100, Peter Eisentraut wrote in > On 2021-02-17 05:05, Kyotaro Horiguchi wrote: > > The commit fe61df7f82 shot down this. > > This patch allows a new GUC ssl_crl_dir and a new libpq connection > > option sslcrldir to specify CRL director

Re: proposal - psql - use pager for \watch command

2021-02-18 Thread Pavel Stehule
Hi út 16. 2. 2021 v 2:49 odesílatel Thomas Munro napsal: > On Fri, Jan 8, 2021 at 10:36 PM Pavel Stehule > wrote: > > ne 19. 4. 2020 v 19:27 odesílatel Pavel Stehule > napsal: > >> last week I finished pspg 3.0 https://github.com/okbob/pspg . pspg now > supports pipes, named pipes very well. T

Re: ERROR: "ft1" is of the wrong type.

2021-02-18 Thread Kyotaro Horiguchi
At Thu, 18 Feb 2021 16:27:23 +0900, Michael Paquier wrote in > On Tue, Feb 16, 2021 at 06:14:15PM +0900, Kyotaro Horiguchi wrote: > > The attached is just fixing that. I tried to make it generic but > > didn't find a clean and translatable way. > > > > Also I found that only three cases in the

Re: repeated decoding of prepared transactions

2021-02-18 Thread Amit Kapila
On Tue, Feb 16, 2021 at 9:43 AM Amit Kapila wrote: > > On Thu, Feb 11, 2021 at 4:06 PM Amit Kapila wrote: > > > > On Mon, Feb 8, 2021 at 2:01 PM Markus Wanner > > wrote: > > > > > Now, coming back to the restart case where the prepared transaction > > can be sent again by the publisher. I unders

Re: [HACKERS] logical decoding of two-phase transactions

2021-02-18 Thread Amit Kapila
On Thu, Feb 18, 2021 at 5:48 AM Peter Smith wrote: > > Please find attached the new patch set v41* > I see one issue here. Currently, when we create a subscription, we first launch apply-worker and create the main apply worker slot and then launch table sync workers as required. Now, assume, the

RE: A reloption for partitioned tables - parallel_workers

2021-02-18 Thread Hou, Zhijie
> hi, > > Here we go, my first patch... solves > https://www.postgresql.org/message-id/7d6fdc20-857c-4cbe-ae2e-c0ff9520 > e...@www.fastmail.com > Hi, partitioned_table_reloptions(Datum reloptions, bool validate) { + static const relopt_parse_elt tab[] = { + {"parallel_worke

Re: [PATCH] pg_hba.conf error messages for logical replication connections

2021-02-18 Thread Amit Kapila
On Thu, Feb 18, 2021 at 5:59 AM Euler Taveira wrote: > > On Wed, Feb 17, 2021, at 8:01 AM, Amit Kapila wrote: > > Before fix > ERROR: could not connect to the publisher: connection to server at > "localhost" (::1), port 5432 failed: FATAL: pg_hba.conf rejects > replication connection for host ":

Re: A reloption for partitioned tables - parallel_workers

2021-02-18 Thread Amit Langote
On Thu, Feb 18, 2021 at 6:06 PM Hou, Zhijie wrote: > > Here we go, my first patch... solves > > https://www.postgresql.org/message-id/7d6fdc20-857c-4cbe-ae2e-c0ff9520 > > e...@www.fastmail.com > > > > Hi, > > partitioned_table_reloptions(Datum reloptions, bool validate) > { > + static const

Allow batched insert during cross-partition updates

2021-02-18 Thread Amit Langote
Based on the discussion at: https://www.postgresql.org/message-id/6929d485-2d2a-da46-3681-4a400a3d794f%40enterprisedb.com I'm posting the patch for $subject here in this new thread and I'll add it to the next CF per Tomas' advice. With 927f453a94106 committed earlier today, we limit insert batch

Re: Allow batched insert during cross-partition updates

2021-02-18 Thread Amit Langote
On Thu, Feb 18, 2021 at 6:52 PM Amit Langote wrote: > > Based on the discussion at: > > https://www.postgresql.org/message-id/6929d485-2d2a-da46-3681-4a400a3d794f%40enterprisedb.com > > I'm posting the patch for $subject here in this new thread and I'll > add it to the next CF per Tomas' advice.

Re: a misbehavior of partition row movement (?)

2021-02-18 Thread Rahila Syed
Hi Amit, > > Here is an updated version of the patch with some cosmetic changes > from the previous version. I moved the code being added to > AfterTriggerSaveEvent() and ExecUpdate() into separate subroutines to > improve readability, hopefully. > > I tested these patches. It works as expected

Re: Extensions not dumped when --schema is used

2021-02-18 Thread Guillaume Lelarge
Le mar. 26 janv. 2021 à 13:42, Guillaume Lelarge a écrit : > Le mar. 26 janv. 2021 à 13:41, Guillaume Lelarge > a écrit : > >> Le mar. 26 janv. 2021 à 05:10, Julien Rouhaud a >> écrit : >> >>> On Mon, Jan 25, 2021 at 9:34 PM Guillaume Lelarge >>> wrote: >>> > >>> > "Anytime soon" was a long lo

Re: Some regular-expression performance hacking

2021-02-18 Thread Joel Jacobson
On Wed, Feb 17, 2021, at 22:00, Tom Lane wrote: > Attached is an updated patch series; it's rebased over 4e703d671 > which took care of some not-really-related fixes, and I made a > pass of cleanup and comment improvements. I think this is pretty > much ready to commit, unless you want to do more

Re: 64-bit XIDs in deleted nbtree pages

2021-02-18 Thread Masahiko Sawada
On Wed, Feb 17, 2021 at 5:41 AM Peter Geoghegan wrote: > > On Tue, Feb 16, 2021 at 11:35 AM Peter Geoghegan wrote: > > Isn't btvacuumcleanup() (or any other amvacuumcleanup() routine) > > entitled to rely on the bulk delete stats being set in the way I've > > described? I assumed that that was ok

Re: Parallel INSERT (INTO ... SELECT ...)

2021-02-18 Thread Greg Nancarrow
On Thu, Feb 18, 2021 at 4:35 PM Amit Langote wrote: > > Looking at this again, I am a bit concerned about going over the whole > partition tree *twice* when making a parallel plan for insert into > partitioned tables. Maybe we should do what you did in your first > attempt a slightly differently

Re: Some regular-expression performance hacking

2021-02-18 Thread Joel Jacobson
On Thu, Feb 18, 2021, at 11:30, Joel Jacobson wrote: >SELECT * FROM vdeviations; >-[ RECORD 1 >]+--- >pattern | \.(ac|com\.ac|edu\.ac|gov\.ac|net\.ac|mi ... 100497 chars >... abs\.org|

Re: 64-bit XIDs in deleted nbtree pages

2021-02-18 Thread Masahiko Sawada
On Tue, Feb 16, 2021 at 12:26 PM Peter Geoghegan wrote: > > On Mon, Feb 15, 2021 at 3:15 AM Masahiko Sawada wrote: > > Yes. I think this would simplify the problem by resolving almost all > > problems related to indefinite deferring page recycle. > > > > We will be able to recycle almost all just

Re: Implementing Incremental View Maintenance

2021-02-18 Thread Andy Fan
On Tue, Feb 16, 2021 at 9:33 AM Yugo NAGATA wrote: > Hi, > > Attached is a rebased patch (v22a). > Thanks for the patch. Will you think posting a patch with the latest commit at that time is helpful? If so, when others want to review it, they know which commit to apply the patch without asking

Re: Printing LSN made easy

2021-02-18 Thread Peter Eisentraut
Here is an updated patch that just introduces LSN_FORMAT_ARGS(). I think the result is quite pleasant. -- Peter Eisentraut 2ndQuadrant, an EDB company https://www.2ndquadrant.com/ From 91d831518c8a71b010bdc835caacaedb20ffb896 Mon Sep 17 00:00:00 2001 From: Peter Eisentraut Date: Thu, 18 Feb 20

Disallow SSL compression?

2021-02-18 Thread Daniel Gustafsson
A few years ago we discussed whether to disable SSL compression [0] which ended up with it being off by default combined with a recommendation against it in the docs. OpenSSL themselves disabled SSL compression by default in 2016 in 1.1.0 with distros often having had it disabled for a long while

Re: Keep notnullattrs in RelOptInfo (Was part of UniqueKey patch series)

2021-02-18 Thread Andy Fan
On Tue, Feb 16, 2021 at 12:01 PM David Rowley wrote: > On Fri, 12 Feb 2021 at 15:18, Andy Fan wrote: > > > > On Fri, Feb 12, 2021 at 9:02 AM David Rowley > wrote: > >> The reason I don't really like this is that it really depends where > >> you want to use RelOptInfo.notnullattrs. If someone w

Re: Printing LSN made easy

2021-02-18 Thread Ashutosh Bapat
On Thu, Feb 18, 2021 at 6:19 PM Peter Eisentraut < peter.eisentr...@2ndquadrant.com> wrote: > Here is an updated patch that just introduces LSN_FORMAT_ARGS(). I > think the result is quite pleasant. > Thanks a lot Peter for producing this patch. I am fine with it. The way this is defined someone

Re: ERROR: invalid spinlock number: 0

2021-02-18 Thread Fujii Masao
On 2021/02/17 13:52, Michael Paquier wrote: On Tue, Feb 16, 2021 at 11:47:52PM +0900, Fujii Masao wrote: On 2021/02/16 15:50, Michael Paquier wrote: + /* +* Read "writtenUpto" without holding a spinlock. So it may not be +* consistent with other WAL receiver's shared variables prot

Re: Problem with accessing TOAST data in stored procedures

2021-02-18 Thread Konstantin Knizhnik
On 19.08.2020 22:20, Pavel Stehule wrote: st 19. 8. 2020 v 20:59 odesílatel Konstantin Knizhnik mailto:k.knizh...@postgrespro.ru>> napsal: On 19.08.2020 21:50, Pavel Stehule wrote: Hi st 19. 8. 2020 v 19:22 odesílatel Konstantin Knizhnik mailto:k.knizh...@postgrespro.ru

Re: Improvements and additions to COPY progress reporting

2021-02-18 Thread Matthias van de Meent
Hi, Thank you all for the suggestions. PFA version 8 of the patchset, in which I have applied most of your comments. Unless explicitly named below, I have applied the suggestions. On Mon, 15 Feb 2021 at 17:07, Tomas Vondra wrote: > > - The blocks in copyfrom.cc/copyto.c should be reworked - I d

cursor sensitivity misunderstanding

2021-02-18 Thread Peter Eisentraut
I think our documentation is mistaken about what it means for a cursor to be "sensitive" or "insensitive". The definition in SQL:2016 is: A change to SQL-data is said to be independent of a cursor CR if and only if it is not made by an or a that is positioned on CR. A change

Re: Finding cause of test fails on the cfbot site

2021-02-18 Thread Andrew Dunstan
On 2/17/21 3:42 PM, Thomas Munro wrote: > On Thu, Feb 18, 2021 at 9:18 AM Andrew Dunstan wrote: >> On 2/17/21 11:06 AM, Tom Lane wrote: >>> Peter Smith writes: I saw that one of our commitfest entries (32/2914) is recently reporting a fail on the cfbot site [1]. I thought this was all

Re: Extensibility of the PostgreSQL wire protocol

2021-02-18 Thread Jan Wieck
Attached are a first patch and a functioning extension that implements a telnet protocol server. The extension needs to be loaded via shared_preload_libraries and configured for a port number and listen_addresses as follows: shared_preload_libraries = 'telnet_srv' telnet_srv.listen_addresses = '

Re: cursor sensitivity misunderstanding

2021-02-18 Thread David G. Johnston
On Thu, Feb 18, 2021 at 9:00 AM Peter Eisentraut < peter.eisentr...@enterprisedb.com> wrote: > > And that seems definitely wrong. Declaring c1 in the above example as > FOR UPDATE or FOR SHARE does not change the result. I think this > discussion is mixing up the concept of cursor sensitivity wi

Re: [Proposal] Page Compression for OLTP

2021-02-18 Thread David Fetter
On Tue, Feb 16, 2021 at 11:15:36PM +0800, chenhj wrote: > At 2021-02-16 21:51:14, "Daniel Gustafsson" wrote: > > >> On 16 Feb 2021, at 15:45, chenhj wrote: > > > >> I want to know whether this patch can be accepted by the community, that > >> is, whether it is necessary for me to continue worki

Re: Problem with accessing TOAST data in stored procedures

2021-02-18 Thread Pavel Stehule
čt 18. 2. 2021 v 16:01 odesílatel Konstantin Knizhnik < k.knizh...@postgrespro.ru> napsal: > > > On 19.08.2020 22:20, Pavel Stehule wrote: > > > > st 19. 8. 2020 v 20:59 odesílatel Konstantin Knizhnik < > k.knizh...@postgrespro.ru> napsal: > >> >> >> On 19.08.2020 21:50, Pavel Stehule wrote: >> >>

Re: Problem with accessing TOAST data in stored procedures

2021-02-18 Thread Pavel Stehule
čt 18. 2. 2021 v 18:10 odesílatel Pavel Stehule napsal: > > > čt 18. 2. 2021 v 16:01 odesílatel Konstantin Knizhnik < > k.knizh...@postgrespro.ru> napsal: > >> >> >> On 19.08.2020 22:20, Pavel Stehule wrote: >> >> >> >> st 19. 8. 2020 v 20:59 odesílatel Konstantin Knizhnik < >> k.knizh...@postgre

Re: Some regular-expression performance hacking

2021-02-18 Thread Tom Lane
"Joel Jacobson" writes: >> I've produced a new dataset which now also includes the regex flags (if >> any) used for each subject applied to a pattern. Again, thanks for collecting this data! I'm a little confused about how you produced the results in the "tests" table, though. It sort of looks

Re: cursor sensitivity misunderstanding

2021-02-18 Thread Peter Eisentraut
On 18.02.21 17:11, David G. Johnston wrote: The OP was doing a course based on Oracle and was confused regarding our behavior.  The documentation failed to help me provide a useful response, so I'd agree there is something here that needs reworking if not outright fixing. According to the p

Re: Some regular-expression performance hacking

2021-02-18 Thread Tom Lane
I thought it was worth looking a little more closely at the error cases in this set of tests, as a form of competition analysis versus Javascript's regex engine. I ran through the cases that gave errors, and pinned down exactly what was causing the error for as many cases as I could. (These resul

many-to-many problem

2021-02-18 Thread Amichai Amar
Hi, I’d like to use PG as an analytics engine on multiple separately created read-only datasets. The datasets, in my case, are independently created by bringing up a local PG instance, populating a table, and shutting down the PG instance. I do have control over both the creation and the query pro

Re: Some regular-expression performance hacking

2021-02-18 Thread Joel Jacobson
On Thu, Feb 18, 2021, at 19:10, Tom Lane wrote: > "Joel Jacobson" writes: > >> I've produced a new dataset which now also includes the regex flags (if > >> any) used for each subject applied to a pattern. > > Again, thanks for collecting this data! I'm a little confused about > how you produced

Re: Support for NSS as a libpq TLS backend

2021-02-18 Thread Jacob Champion
On Wed, 2021-02-17 at 22:35 +0100, Daniel Gustafsson wrote: > Attached is a rebase on top of this and the recent cryptohash changes to pass > in buffer lengths to the _final function. On top of that, I fixed up and > expanded the documentation, improved SCRAM handling (by using NSS digest > operat

Re: Some regular-expression performance hacking

2021-02-18 Thread Joel Jacobson
On Thu, Feb 18, 2021, at 20:58, Joel Jacobson wrote: >Like you said earlier, perhaps the regex engine has been optimized enough for >this time. >If not, you want to investigate an additional idea, In the above sentence, I meant "you _may_ want to". I'm not at all sure these idea are applicable in

Re: Some regular-expression performance hacking

2021-02-18 Thread Tom Lane
"Joel Jacobson" writes: > Let's see if I can explain the idea: > One of the problems with representing regexes with large bracket range > expressions, like [a-z], > is you get an explosion of edges, if the model can only represent state > transitions for single characters. > If we could instead

Re: Some regular-expression performance hacking

2021-02-18 Thread Joel Jacobson
On Thu, Feb 18, 2021, at 21:44, Tom Lane wrote: >Hmm ... I might be misunderstanding, but I think our engine already >does a version of this. See the discussion of "colors" in >src/backend/regex/README. Thanks, I will read it with great interest. >Maybe. In practice the actual scanning tends to

Re: Improvements and additions to COPY progress reporting

2021-02-18 Thread Tomas Vondra
On 2/18/21 4:46 PM, Matthias van de Meent wrote: > Hi, > > Thank you all for the suggestions. PFA version 8 of the patchset, in > which I have applied most of your comments. Unless explicitly named > below, I have applied the suggestions. > Thanks. > > On Mon, 15 Feb 2021 at 17:07, Tomas Vo

Re: [POC] verifying UTF-8 using SIMD instructions

2021-02-18 Thread John Naylor
On Mon, Feb 15, 2021 at 9:32 PM John Naylor wrote: > > On Mon, Feb 15, 2021 at 9:18 AM Heikki Linnakangas wrote: > > > > I'm guessing that's because the unaligned access in check_ascii() is > > expensive on this platform. > Some possible remedies: > 3) #ifdef out the ascii check for 32-bit plat

Re: Printing LSN made easy

2021-02-18 Thread Kyotaro Horiguchi
At Thu, 18 Feb 2021 18:51:37 +0530, Ashutosh Bapat wrote in > On Thu, Feb 18, 2021 at 6:19 PM Peter Eisentraut < > peter.eisentr...@2ndquadrant.com> wrote: > > > Here is an updated patch that just introduces LSN_FORMAT_ARGS(). I > > think the result is quite pleasant. > > > > Thanks a lot Pet

Re: Implementing Incremental View Maintenance

2021-02-18 Thread Yugo NAGATA
On Thu, 18 Feb 2021 19:38:44 +0800 Andy Fan wrote: > On Tue, Feb 16, 2021 at 9:33 AM Yugo NAGATA wrote: > > > Hi, > > > > Attached is a rebased patch (v22a). > > > > Thanks for the patch. Will you think posting a patch with the latest commit > at that > time is helpful? If so, when others want

Re: PoC Refactor AM analyse API

2021-02-18 Thread Denis Smirnov
Thanks for your review, Heikki. I have made the changes you have requested. 1. All modifications interconnected with column projection were reverted (they should be added in https://commitfest.postgresql.org/31/2922 if the current patch would be merged one day). 2. I have returned PROGRESS_ANAL

Re: PoC Refactor AM analyse API

2021-02-18 Thread Zhihong Yu
Hi, + *totalrows = floor((liverows / bs.m) * totalblocks + 0.5); Is the above equivalent to: + *totalrows = ceil((liverows / bs.m) * totalblocks); For compare_rows(), it seems the computation of oa and ob can be delayed to when ba == bb (after the first two if statements). Cheers

Re: Two patches to speed up pg_rewind.

2021-02-18 Thread Paul Guo
Refactored the code a bit along with fixes. Manually tested them on centos & Ubuntu (the later has copy_file_range()) For the first patch, actually I have some concerns. My assumption is that the target pg_data directory should be fsync-ed already. This should be correct normally but there is one

Re: Freeze the inserted tuples during CTAS?

2021-02-18 Thread Paul Guo
On Jan 27, 2021, at 5:33 PM, Darafei Komяpa Praliaskouski mailto:m...@komzpa.net>> wrote: Hi, I confirm that my analytic workflows often do the CTAS and VACUUM of the relation right after, before the index creation, to mark stuff as all-visible for IOS to work. Freezing and marking as visibl

Re: repeated decoding of prepared transactions

2021-02-18 Thread Ajin Cherian
On Tue, Feb 16, 2021 at 3:13 PM Amit Kapila wrote: > > After further thinking on this problem and some off-list discussions > with Ajin, there appears to be another way to solve the above problem > by which we can avoid resending the prepare after restart if it has > already been processed by the

Re: a misbehavior of partition row movement (?)

2021-02-18 Thread Amit Langote
Hi Rahila, Thanks for the review. On Thu, Feb 18, 2021 at 7:08 PM Rahila Syed wrote: >> Here is an updated version of the patch with some cosmetic changes >> from the previous version. I moved the code being added to >> AfterTriggerSaveEvent() and ExecUpdate() into separate subroutines to >> im

Re: [PoC] Non-volatile WAL buffer

2021-02-18 Thread Tomas Vondra
On 1/22/21 5:04 PM, Konstantin Knizhnik wrote: > ... > > I have heard from several DBMS experts that appearance of huge and > cheap non-volatile memory can make a revolution in database system > architecture. If all database can fit in non-volatile memory, then we > do not need buffers, WAL, ...> >

RE: Parallel INSERT (INTO ... SELECT ...)

2021-02-18 Thread tsunakawa.ta...@fujitsu.com
From: Greg Nancarrow -- On Mon, Jan 25, 2021 at 10:23 AM tsunakawa.ta...@fujitsu.com wrote: > (8) > + /* > +* If the trigger type is RI_TRIGGER_FK, this indicates a FK > exists in > +* the relation, a

Re: cryptohash: missing locking functions for OpenSSL <= 1.0.2?

2021-02-18 Thread Michael Paquier
On Thu, Feb 18, 2021 at 11:04:05AM +0900, Michael Paquier wrote: > We have the code in place to properly initialize the crypto locking in > libpq with ENABLE_THREAD_SAFETY, but the root of the issue is that the > SSL and crypto initializations are grouped together. What we need to > do here is to

Re: progress reporting for partitioned REINDEX

2021-02-18 Thread Michael Paquier
On Thu, Feb 18, 2021 at 02:17:00PM +0900, Michael Paquier wrote: > I have no issues with documenting more precisely on which commands > partitions_total and partitions_done apply currently, by citing the > commands where these are effective. We do that for index_relid for > instance. Please find

Re: Printing LSN made easy

2021-02-18 Thread Michael Paquier
On Fri, Feb 19, 2021 at 10:54:05AM +0900, Kyotaro Horiguchi wrote: > At Thu, 18 Feb 2021 18:51:37 +0530, Ashutosh Bapat > wrote in >> On Thu, Feb 18, 2021 at 6:19 PM Peter Eisentraut < >> peter.eisentr...@2ndquadrant.com> wrote: >>> Here is an updated patch that just introduces LSN_FORMAT_ARGS()

Re: progress reporting for partitioned REINDEX

2021-02-18 Thread Justin Pryzby
On Fri, Feb 19, 2021 at 03:06:04PM +0900, Michael Paquier wrote: > On Thu, Feb 18, 2021 at 02:17:00PM +0900, Michael Paquier wrote: > > I have no issues with documenting more precisely on which commands > > partitions_total and partitions_done apply currently, by citing the > > commands where these

Re: 64-bit XIDs in deleted nbtree pages

2021-02-18 Thread Masahiko Sawada
On Fri, Feb 19, 2021 at 3:12 PM Peter Geoghegan wrote: > > On Thu, Feb 18, 2021 at 3:13 AM Masahiko Sawada wrote: > > Agreed. Thanks for your explanation. > > Attached is v5, which has some of the changes I talked about. Changes > from v4 include: > > * Now only updates metapage during btvacuumcl

pg_config_h.in not up-to-date

2021-02-18 Thread Antonin Houska
When I run "autoreconf" on the master branch, git generates the diff below. Shouldn't it just be applied? I suppose someone changed configure.ac and forgot to update the generated file. -- Antonin Houska Web: https://www.cybertec-postgresql.com diff --git a/src/include/pg_config.h.in b/src/inclu

Re: pg_config_h.in not up-to-date

2021-02-18 Thread Tom Lane
Antonin Houska writes: > When I run "autoreconf" on the master branch, git generates the diff > below. Shouldn't it just be applied? I suppose someone changed configure.ac > and forgot to update the generated file. Yeah, looks like fe61df7f8 is at fault. Michael? regards

Re: Problem with accessing TOAST data in stored procedures

2021-02-18 Thread Konstantin Knizhnik
On 18.02.2021 20:10, Pavel Stehule wrote: This has a negative impact on performance - and a lot of users use procedures without transaction control. So it doesn't look like a good solution. I am more concentrated on the Pg 14 release, where the work with SPI is redesigned, and I hope so thi

Re: pg_config_h.in not up-to-date

2021-02-18 Thread Michael Paquier
On Fri, Feb 19, 2021 at 01:42:38AM -0500, Tom Lane wrote: > Antonin Houska writes: >> When I run "autoreconf" on the master branch, git generates the diff >> below. Shouldn't it just be applied? I suppose someone changed configure.ac >> and forgot to update the generated file. > > Yeah, looks lik

Re: Problem with accessing TOAST data in stored procedures

2021-02-18 Thread Pavel Stehule
pá 19. 2. 2021 v 7:51 odesílatel Konstantin Knizhnik < k.knizh...@postgrespro.ru> napsal: > > > On 18.02.2021 20:10, Pavel Stehule wrote: > > This has a negative impact on performance - and a lot of users use > procedures without transaction control. So it doesn't look like a good > solution. > >

Re: Problem with accessing TOAST data in stored procedures

2021-02-18 Thread Konstantin Knizhnik
I am sorry, maybe my reply was not (is not) correct - this issue was reported four months ago, and now I think more about your words about ATX, and I have no idea, how much it is related to community pg or to pgpro. I am sure so implementation of autonomous transactions is pretty hard, but

Re: pg_config_h.in not up-to-date

2021-02-18 Thread Tom Lane
Michael Paquier writes: > Indeed, thanks. It looks like a "git add" that was fat-fingered. I > would like to make things more consistent with the attached. +1, but I think the first period in this comment is redundant: + AC_DEFINE([USE_OPENSSL], 1, [Define to 1 to build with OpenSSL support.

Re: A reloption for partitioned tables - parallel_workers

2021-02-18 Thread Amit Langote
On Tue, Feb 16, 2021 at 3:05 PM Amit Langote wrote: > On Tue, Feb 16, 2021 at 1:35 AM Seamus Abshere wrote: > > Here we go, my first patch... solves > > https://www.postgresql.org/message-id/7d6fdc20-857c-4cbe-ae2e-c0ff9520e...@www.fastmail.com > > Thanks for sending the patch here. > > It seems

Re: pg_config_h.in not up-to-date

2021-02-18 Thread Michael Paquier
On Fri, Feb 19, 2021 at 02:21:21AM -0500, Tom Lane wrote: > Michael Paquier writes: > > Indeed, thanks. It looks like a "git add" that was fat-fingered. I > > would like to make things more consistent with the attached. > > +1, but I think the first period in this comment is redundant: > > +

Re: Tid scan improvements

2021-02-18 Thread David Rowley
On Thu, 18 Feb 2021 at 09:45, David Rowley wrote: > > On Wed, 17 Feb 2021 at 11:05, Andres Freund wrote: > > Architecturally it feels like this is something that really belongs more > > into plan time? > > Possibly. It would mean TidOpExpr would have to become a Node type. > TID Range scan is rea

Re: Problem with accessing TOAST data in stored procedures

2021-02-18 Thread Konstantin Knizhnik
On 19.02.2021 10:14, Pavel Stehule wrote: pá 19. 2. 2021 v 7:51 odesílatel Konstantin Knizhnik mailto:k.knizh...@postgrespro.ru>> napsal: On 18.02.2021 20:10, Pavel Stehule wrote: This has a negative impact on performance - and a lot of users use procedures without transacti

Re: Problem with accessing TOAST data in stored procedures

2021-02-18 Thread Pavel Stehule
pá 19. 2. 2021 v 8:17 odesílatel Konstantin Knizhnik < k.knizh...@postgrespro.ru> napsal: > > I am sorry, maybe my reply was not (is not) correct - this issue was > reported four months ago, and now I think more about your words about ATX, > and I have no idea, how much it is related to community

Re: Problem with accessing TOAST data in stored procedures

2021-02-18 Thread Pavel Stehule
pá 19. 2. 2021 v 8:39 odesílatel Konstantin Knizhnik < k.knizh...@postgrespro.ru> napsal: > > > On 19.02.2021 10:14, Pavel Stehule wrote: > > > > pá 19. 2. 2021 v 7:51 odesílatel Konstantin Knizhnik < > k.knizh...@postgrespro.ru> napsal: > >> >> >> On 18.02.2021 20:10, Pavel Stehule wrote: >> >> T

Re: PoC Refactor AM analyse API

2021-02-18 Thread Denis Smirnov
Hello, Zhihong. Thanks for your comments. 1. I am afraid that an equivalence of "floor(val + 0.5)" to "cell(val)" is incorrect: "floor(2.1 + 0.5)" returns 2 while "cell(2.1)" returns 3. We can’t use such replacement, as you have suggested. 2. >> For compare_rows(), it seems the computation o