Re: Some oversights in query_id calculation

2021-05-04 Thread Julien Rouhaud
On Mon, May 03, 2021 at 02:59:42PM -0400, Bruce Momjian wrote: > On Sun, May 2, 2021 at 12:27:37PM +0800, Julien Rouhaud wrote: > > Hi Aleksander, > > > > On Wed, Apr 28, 2021 at 03:22:39PM +0300, Aleksander Alekseev wrote: > > > Hi Julien, > > > > > > > You should see failures doing a

Re: Some oversights in query_id calculation

2021-05-03 Thread Bruce Momjian
On Sun, May 2, 2021 at 12:27:37PM +0800, Julien Rouhaud wrote: > Hi Aleksander, > > On Wed, Apr 28, 2021 at 03:22:39PM +0300, Aleksander Alekseev wrote: > > Hi Julien, > > > > > You should see failures doing a check-world or simply a make -C > > > contrib/pg_stat_statements check > > > >

Re: Some oversights in query_id calculation

2021-05-01 Thread Julien Rouhaud
Hi Aleksander, On Wed, Apr 28, 2021 at 03:22:39PM +0300, Aleksander Alekseev wrote: > Hi Julien, > > > You should see failures doing a check-world or simply a make -C > > contrib/pg_stat_statements check > > Sorry, my bad. I was running make check-world, but did it with -j4 flag > which was a

Re: Some oversights in query_id calculation

2021-04-28 Thread Aleksander Alekseev
Hi Julien, > You should see failures doing a check-world or simply a make -C > contrib/pg_stat_statements check Sorry, my bad. I was running make check-world, but did it with -j4 flag which was a mistake. The patch is OK. On Wed, Apr 28, 2021 at 1:27 PM Julien Rouhaud wrote: > Hi

Re: Some oversights in query_id calculation

2021-04-28 Thread Julien Rouhaud
Hi Aleksander, On Wed, Apr 28, 2021 at 01:19:36PM +0300, Aleksander Alekseev wrote: > Hi Julien, > > > I'm attaching a patch that fixes those, with regression tests to reproduce > > each > > problem. > > I believe something could be not quite right with the patch. Here is what I > did: > > $

Re: Some oversights in query_id calculation

2021-04-28 Thread Aleksander Alekseev
Hi Julien, > I'm attaching a patch that fixes those, with regression tests to reproduce > each > problem. I believe something could be not quite right with the patch. Here is what I did: $ git apply ... # revert the changes in the code but keep the new tests $ git checkout

Some oversights in query_id calculation

2021-04-25 Thread Julien Rouhaud
455af7ce Mon Sep 17 00:00:00 2001 From: Julien Rouhaud Date: Sun, 25 Apr 2021 15:49:32 +0800 Subject: [PATCH v1] Fix some oversights in query_id calculation. --- .../expected/pg_stat_statements.out | 151 ++ .../sql/pg_stat_statements.sql| 52 ++ src