pg.ddx.io pgsql-hackers@postgresql.org mailing list archive
help / color / mirror / Atom feed[PATCH] pgbench tap tests fail if the path contains a perl special character
22+ messages / 8 participants
[nested] [flat]
* [PATCH] pgbench tap tests fail if the path contains a perl special character
@ 2019-01-18 17:15 Raúl Marín Rodríguez <rmrodriguez@carto.com>
2019-01-18 17:35 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Tom Lane <tgl@sss.pgh.pa.us>
0 siblings, 1 reply; 22+ messages in thread
From: Raúl Marín Rodríguez @ 2019-01-18 17:15 UTC (permalink / raw)
To: pgsql-hackers
Hi,
As mentioned in the subject, if the $PATH where you are building postgres
contains a perl's special character [1], for example a `+`, pgbench's tap
tests will fail.
The outputs looks something like this (full at [2]):
```
# Failed test 'file name format'
# at t/001_pgbench_with_server.pl line 805.
# Failed test 'file name format'
# at t/001_pgbench_with_server.pl line 805.
# Looks like you failed 2 tests of 312.
t/001_pgbench_with_server.pl ..
Dubious, test returned 2 (wstat 512, 0x200)
Failed 2/312 subtests
t/002_pgbench_no_server.pl .... ok
```
This error affects both PG11 and master, and a way to reproduce it is by
renaming the source folder from `postgresql` to `postgresql+XXX`.
I'm attaching a patch to fix this by using quotemeta [3]. It's generated
from master and also applies to REL_11_STABLE.
1 - http://jkorpela.fi/perl/regexp.html
2 - https://launchpadlibrarian.net/406634533/buildlog_ubuntu-xenial-amd64.postgresql-11_11.1.1+carto-2_B...
3 - https://perldoc.perl.org/functions/quotemeta.html
Regards,
--
Raúl Marín Rodríguez
carto.com
Attachments:
[text/x-patch] pgbench_prefix-v1.patch (741B, ../../CAM6_UM6dGdU39PKAC24T+HD9ouy0jLN9vH6163K8QEEzr__iZw@mail.gmail.com/2-pgbench_prefix-v1.patch)
download | inline diff:
diff --git a/src/bin/pgbench/t/001_pgbench_with_server.pl b/src/bin/pgbench/t/001_pgbench_with_server.pl
index c87748086a..eb126d2525 100644
--- a/src/bin/pgbench/t/001_pgbench_with_server.pl
+++ b/src/bin/pgbench/t/001_pgbench_with_server.pl
@@ -879,9 +879,10 @@ sub check_pgbench_logs
my ($prefix, $nb, $min, $max, $re) = @_;
- my @logs = glob "$prefix.*";
+ my @quoted_prefix = quotemeta($prefix);
+ my @logs = glob "@quoted_prefix.*";
ok(@logs == $nb, "number of log files");
- ok(grep(/^$prefix\.\d+(\.\d+)?$/, @logs) == $nb, "file name format");
+ ok(grep(/^@quoted_prefix\.\d+(\.\d+)?$/, @logs) == $nb, "Failed to match file format. Looked for (@quoted_prefix) in (@logs)");
my $log_number = 0;
for my $log (sort @logs)
^ permalink raw reply [nested|flat] 22+ messages in thread
* Re: [PATCH] pgbench tap tests fail if the path contains a perl special character
2019-01-18 17:15 [PATCH] pgbench tap tests fail if the path contains a perl special character Raúl Marín Rodríguez <rmrodriguez@carto.com>
@ 2019-01-18 17:35 ` Tom Lane <tgl@sss.pgh.pa.us>
2019-01-18 17:54 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Raúl Marín Rodríguez <rmrodriguez@carto.com>
0 siblings, 1 reply; 22+ messages in thread
From: Tom Lane @ 2019-01-18 17:35 UTC (permalink / raw)
To: Raúl Marín Rodríguez <rmrodriguez@carto.com>; +Cc: pgsql-hackers
=?UTF-8?B?UmHDumwgTWFyw61uIFJvZHLDrWd1ZXo=?= <rmrodriguez@carto.com> writes:
> As mentioned in the subject, if the $PATH where you are building postgres
> contains a perl's special character [1], for example a `+`, pgbench's tap
> tests will fail.
Fun. But is that really the only place that fails?
> I'm attaching a patch to fix this by using quotemeta [3].
This does not look right to me: glob has a different set of special
characters than regexes, no?
regards, tom lane
^ permalink raw reply [nested|flat] 22+ messages in thread
* Re: [PATCH] pgbench tap tests fail if the path contains a perl special character
2019-01-18 17:15 [PATCH] pgbench tap tests fail if the path contains a perl special character Raúl Marín Rodríguez <rmrodriguez@carto.com>
2019-01-18 17:35 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Tom Lane <tgl@sss.pgh.pa.us>
@ 2019-01-18 17:54 ` Raúl Marín Rodríguez <rmrodriguez@carto.com>
2019-01-18 19:27 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Alvaro Herrera <alvherre@2ndquadrant.com>
0 siblings, 1 reply; 22+ messages in thread
From: Raúl Marín Rodríguez @ 2019-01-18 17:54 UTC (permalink / raw)
To: Tom Lane <tgl@sss.pgh.pa.us>; +Cc: pgsql-hackers
Hi,
> Fun. But is that really the only place that fails?
Yes, other than this it builds flawlessly.
> This does not look right to me: glob has a different set of special
> characters than regexes, no?
The issue itself manifests in the grep call, not in glob, but I tried
using `quotemeta` only for grep without success (probably because I didn't
know how to properly call `quotemeta` over @logs.
This is the first time I do anything in perl beyond adding some simple tap
tests, so this could be me not knowing how to deal with it properly. I used
this small script to understand perl's behaviour:
```
$ cat a.pl
#!/usr/bin/perl
use strict;
use warnings;
my @prefix = '~/postgres+XXXX/001_pgbench_log_2.20745';
my @quoted_prefix = quotemeta(@prefix);
my @found1 = grep(/^@prefix\.\d+(\.\d+)?$/, "@prefix.20745");
my @found2 = grep(/^@prefix\.\d+(\.\d+)?$/, "@quoted_prefix.20745");
my @found3 = grep(/^@quoted_prefix\.\d+(\.\d+)?$/, "@prefix.20745");
my @found4 = grep(/^@quoted_prefix\.\d+(\.\d+)?$/, "@quoted_prefix.20745");
print "1: @found1\n";
print "2: @found2\n";
print "3: @found3\n";
print "4: @found4\n";
$ perl a.pl
1:
2:
3:
4: 1.20745
```
Regards,
--
Raúl Marín Rodríguez
carto.com
^ permalink raw reply [nested|flat] 22+ messages in thread
* Re: [PATCH] pgbench tap tests fail if the path contains a perl special character
2019-01-18 17:15 [PATCH] pgbench tap tests fail if the path contains a perl special character Raúl Marín Rodríguez <rmrodriguez@carto.com>
2019-01-18 17:35 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Tom Lane <tgl@sss.pgh.pa.us>
2019-01-18 17:54 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Raúl Marín Rodríguez <rmrodriguez@carto.com>
@ 2019-01-18 19:27 ` Alvaro Herrera <alvherre@2ndquadrant.com>
2019-01-18 20:02 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Tom Lane <tgl@sss.pgh.pa.us>
0 siblings, 1 reply; 22+ messages in thread
From: Alvaro Herrera @ 2019-01-18 19:27 UTC (permalink / raw)
To: Raúl Marín Rodríguez <rmrodriguez@carto.com>; +Cc: Tom Lane <tgl@sss.pgh.pa.us>; pgsql-hackers
On 2019-Jan-18, Raúl Marín Rodríguez wrote:
> Hi,
>
> > Fun. But is that really the only place that fails?
>
> Yes, other than this it builds flawlessly.
>
> > This does not look right to me: glob has a different set of special
> > characters than regexes, no?
>
> The issue itself manifests in the grep call, not in glob, but I tried
> using `quotemeta` only for grep without success (probably because I didn't
> know how to properly call `quotemeta` over @logs.
Perhaps you can do
@quoted = map { quotemeta($_) } @logs;
--
Álvaro Herrera https://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services
^ permalink raw reply [nested|flat] 22+ messages in thread
* Re: [PATCH] pgbench tap tests fail if the path contains a perl special character
2019-01-18 17:15 [PATCH] pgbench tap tests fail if the path contains a perl special character Raúl Marín Rodríguez <rmrodriguez@carto.com>
2019-01-18 17:35 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Tom Lane <tgl@sss.pgh.pa.us>
2019-01-18 17:54 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Raúl Marín Rodríguez <rmrodriguez@carto.com>
2019-01-18 19:27 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Alvaro Herrera <alvherre@2ndquadrant.com>
@ 2019-01-18 20:02 ` Tom Lane <tgl@sss.pgh.pa.us>
2019-01-18 20:26 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Tom Lane <tgl@sss.pgh.pa.us>
0 siblings, 1 reply; 22+ messages in thread
From: Tom Lane @ 2019-01-18 20:02 UTC (permalink / raw)
To: Alvaro Herrera <alvherre@2ndquadrant.com>; +Cc: Raúl Marín Rodríguez <rmrodriguez@carto.com>; pgsql-hackers
Alvaro Herrera <alvherre@2ndquadrant.com> writes:
>>> This does not look right to me: glob has a different set of special
>>> characters than regexes, no?
> Perhaps you can do
> @quoted = map { quotemeta($_) } @logs;
Actually, playing with it here, it seems that quotemeta() quotes
anything that's a special character for either purpose (in fact,
it looks like it quotes anything that's not alphanumeric).
So the patch as given should work.
regards, tom lane
^ permalink raw reply [nested|flat] 22+ messages in thread
* Re: [PATCH] pgbench tap tests fail if the path contains a perl special character
2019-01-18 17:15 [PATCH] pgbench tap tests fail if the path contains a perl special character Raúl Marín Rodríguez <rmrodriguez@carto.com>
2019-01-18 17:35 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Tom Lane <tgl@sss.pgh.pa.us>
2019-01-18 17:54 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Raúl Marín Rodríguez <rmrodriguez@carto.com>
2019-01-18 19:27 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Alvaro Herrera <alvherre@2ndquadrant.com>
2019-01-18 20:02 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Tom Lane <tgl@sss.pgh.pa.us>
@ 2019-01-18 20:26 ` Tom Lane <tgl@sss.pgh.pa.us>
2019-01-18 20:56 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Raúl Marín Rodríguez <rmrodriguez@carto.com>
0 siblings, 1 reply; 22+ messages in thread
From: Tom Lane @ 2019-01-18 20:26 UTC (permalink / raw)
To: Alvaro Herrera <alvherre@2ndquadrant.com>; +Cc: Raúl Marín Rodríguez <rmrodriguez@carto.com>; pgsql-hackers
I wrote:
> So the patch as given should work.
... well, modulo the use of "@foo" where you should've used "$foo".
I don't know enough Perl to understand why that seemed to work, but
I do know it's not the right thing here.
Pushed with that correction.
regards, tom lane
^ permalink raw reply [nested|flat] 22+ messages in thread
* Re: [PATCH] pgbench tap tests fail if the path contains a perl special character
2019-01-18 17:15 [PATCH] pgbench tap tests fail if the path contains a perl special character Raúl Marín Rodríguez <rmrodriguez@carto.com>
2019-01-18 17:35 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Tom Lane <tgl@sss.pgh.pa.us>
2019-01-18 17:54 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Raúl Marín Rodríguez <rmrodriguez@carto.com>
2019-01-18 19:27 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Alvaro Herrera <alvherre@2ndquadrant.com>
2019-01-18 20:02 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Tom Lane <tgl@sss.pgh.pa.us>
2019-01-18 20:26 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Tom Lane <tgl@sss.pgh.pa.us>
@ 2019-01-18 20:56 ` Raúl Marín Rodríguez <rmrodriguez@carto.com>
2019-01-21 01:51 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Tom Lane <tgl@sss.pgh.pa.us>
0 siblings, 1 reply; 22+ messages in thread
From: Raúl Marín Rodríguez @ 2019-01-18 20:56 UTC (permalink / raw)
To: Tom Lane <tgl@sss.pgh.pa.us>; +Cc: Alvaro Herrera <alvherre@2ndquadrant.com>; pgsql-hackers
> Pushed with that correction.
Thanks a lot!
--
Raúl Marín Rodríguez
carto.com
^ permalink raw reply [nested|flat] 22+ messages in thread
* Re: [PATCH] pgbench tap tests fail if the path contains a perl special character
2019-01-18 17:15 [PATCH] pgbench tap tests fail if the path contains a perl special character Raúl Marín Rodríguez <rmrodriguez@carto.com>
2019-01-18 17:35 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Tom Lane <tgl@sss.pgh.pa.us>
2019-01-18 17:54 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Raúl Marín Rodríguez <rmrodriguez@carto.com>
2019-01-18 19:27 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Alvaro Herrera <alvherre@2ndquadrant.com>
2019-01-18 20:02 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Tom Lane <tgl@sss.pgh.pa.us>
2019-01-18 20:26 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Tom Lane <tgl@sss.pgh.pa.us>
2019-01-18 20:56 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Raúl Marín Rodríguez <rmrodriguez@carto.com>
@ 2019-01-21 01:51 ` Tom Lane <tgl@sss.pgh.pa.us>
2019-01-21 09:50 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Fabien COELHO <coelho@cri.ensmp.fr>
0 siblings, 1 reply; 22+ messages in thread
From: Tom Lane @ 2019-01-21 01:51 UTC (permalink / raw)
To: Raúl Marín Rodríguez <rmrodriguez@carto.com>; +Cc: Andrew Dunstan <andrew@dunslane.net>; Alvaro Herrera <alvherre@2ndquadrant.com>; pgsql-hackers
=?UTF-8?B?UmHDumwgTWFyw61uIFJvZHLDrWd1ZXo=?= <rmrodriguez@carto.com> writes:
>> Pushed with that correction.
> Thanks a lot!
Hm, so bowerbird (a Windows machine) has been failing the pgbench tests
since this went in, cf
https://buildfarm.postgresql.org/cgi-bin/show_log.pl?nm=bowerbird&dt=2019-01-20%2004%3A57%3A01
I'm just guessing, but I suspect that bowerbird is using a path spec that
includes a backslash directory separator and that's somehow bollixing
things. If so, we might be able to fix it by converting backslashes to
forward slashes before applying quotemeta().
It's also possible that on Windows, "glob" handles backslashes
differently than it does elsewhere, which would be harder to fix
--- that would bring back my original fear that the appropriate
quoting rules are different for "glob" than for regexes.
Andrew, any insight?
regards, tom lane
^ permalink raw reply [nested|flat] 22+ messages in thread
* Re: [PATCH] pgbench tap tests fail if the path contains a perl special character
2019-01-18 17:15 [PATCH] pgbench tap tests fail if the path contains a perl special character Raúl Marín Rodríguez <rmrodriguez@carto.com>
2019-01-18 17:35 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Tom Lane <tgl@sss.pgh.pa.us>
2019-01-18 17:54 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Raúl Marín Rodríguez <rmrodriguez@carto.com>
2019-01-18 19:27 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Alvaro Herrera <alvherre@2ndquadrant.com>
2019-01-18 20:02 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Tom Lane <tgl@sss.pgh.pa.us>
2019-01-18 20:26 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Tom Lane <tgl@sss.pgh.pa.us>
2019-01-18 20:56 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Raúl Marín Rodríguez <rmrodriguez@carto.com>
2019-01-21 01:51 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Tom Lane <tgl@sss.pgh.pa.us>
@ 2019-01-21 09:50 ` Fabien COELHO <coelho@cri.ensmp.fr>
2019-01-21 15:40 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Andrew Dunstan <andrew.dunstan@2ndquadrant.com>
0 siblings, 1 reply; 22+ messages in thread
From: Fabien COELHO @ 2019-01-21 09:50 UTC (permalink / raw)
To: Tom Lane <tgl@sss.pgh.pa.us>; +Cc: Raúl Marín Rodríguez <rmrodriguez@carto.com>; Andrew Dunstan <andrew@dunslane.net>; Alvaro Herrera <alvherre@2ndquadrant.com>; pgsql-hackers
Hello Tom,
> Hm, so bowerbird (a Windows machine) has been failing the pgbench tests
> since this went in, cf
>
> https://buildfarm.postgresql.org/cgi-bin/show_log.pl?nm=bowerbird&dt=2019-01-20%2004%3A57%3A01
>
> I'm just guessing, but I suspect that bowerbird is using a path spec that
> includes a backslash directory separator and that's somehow bollixing
> things.
This point is unclear from the log, where plain slashes are used in the
log prefix path, and furthermore no strange characters appear in the
log prefix path:
# Running: pgbench -n -S -t 50 -c 2 --log --log-prefix=G:/prog/bf/root/HEAD/pgsql.build/src/bin/pgbench/tmp_check/t_001_pgbench_with_server_main_data/001_pgbench_log_2 --sampling-rate=0.5
ok 345 - pgbench logs status (got 0 vs expected 0)
ok 346 - pgbench logs stdout /(?^:select only)/
ok 347 - pgbench logs stdout /(?^:processed: 100/100)/
ok 348 - pgbench logs stderr /(?^:^$)/
not ok 349 - number of log files
> If so, we might be able to fix it by converting backslashes to
> forward slashes before applying quotemeta().
>
> It's also possible that on Windows, "glob" handles backslashes
> differently than it does elsewhere, which would be harder to fix
> --- that would bring back my original fear that the appropriate
> quoting rules are different for "glob" than for regexes.
I'm afraid it could indeed be due to platform-specific behavior, so
testing on the target machine to understand the actual behavior would
help.
I just tested the current version on my laptop within a directory
containing spaces and other misc chars: Pgbench TAP tests are currently
broken in this context on master, and this may be used to a collection of
issues, not just one, eg pgbench function splits parameters on spaces
which breaks if there are spaces in bdir.
I'm going to investigate, possibly over next week-end, so please be
patient.
About windows-specific issues, from File::Glob man page:
""" On DOSISH systems, backslash is a valid directory separator character.
In this case, use of backslash as a quoting character (via GLOB_QUOTE)
interferes with the use of backslash as a directory separator. ... """
It seems to suggest that quotemeta backslash-on-nearly-everything approach
is not appropriate.
--
Fabien.
^ permalink raw reply [nested|flat] 22+ messages in thread
* Re: [PATCH] pgbench tap tests fail if the path contains a perl special character
2019-01-18 17:15 [PATCH] pgbench tap tests fail if the path contains a perl special character Raúl Marín Rodríguez <rmrodriguez@carto.com>
2019-01-18 17:35 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Tom Lane <tgl@sss.pgh.pa.us>
2019-01-18 17:54 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Raúl Marín Rodríguez <rmrodriguez@carto.com>
2019-01-18 19:27 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Alvaro Herrera <alvherre@2ndquadrant.com>
2019-01-18 20:02 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Tom Lane <tgl@sss.pgh.pa.us>
2019-01-18 20:26 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Tom Lane <tgl@sss.pgh.pa.us>
2019-01-18 20:56 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Raúl Marín Rodríguez <rmrodriguez@carto.com>
2019-01-21 01:51 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Tom Lane <tgl@sss.pgh.pa.us>
2019-01-21 09:50 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Fabien COELHO <coelho@cri.ensmp.fr>
@ 2019-01-21 15:40 ` Andrew Dunstan <andrew.dunstan@2ndquadrant.com>
2019-01-21 16:10 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Tom Lane <tgl@sss.pgh.pa.us>
2019-01-21 16:11 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Fabien COELHO <coelho@cri.ensmp.fr>
0 siblings, 2 replies; 22+ messages in thread
From: Andrew Dunstan @ 2019-01-21 15:40 UTC (permalink / raw)
To: Fabien COELHO <coelho@cri.ensmp.fr>; Tom Lane <tgl@sss.pgh.pa.us>; +Cc: Raúl Marín Rodríguez <rmrodriguez@carto.com>; Alvaro Herrera <alvherre@2ndquadrant.com>; pgsql-hackers
On 1/21/19 4:50 AM, Fabien COELHO wrote:
>
> Hello Tom,
>
>> Hm, so bowerbird (a Windows machine) has been failing the pgbench tests
>> since this went in, cf
>>
>> https://buildfarm.postgresql.org/cgi-bin/show_log.pl?nm=bowerbird&dt=2019-01-20%2004%3A57%3A01
>>
>>
>> I'm just guessing, but I suspect that bowerbird is using a path spec
>> that
>> includes a backslash directory separator and that's somehow bollixing
>> things.
>
> This point is unclear from the log, where plain slashes are used in
> the log prefix path, and furthermore no strange characters appear in
> the log prefix path:
>
> # Running: pgbench -n -S -t 50 -c 2 --log
> --log-prefix=G:/prog/bf/root/HEAD/pgsql.build/src/bin/pgbench/tmp_check/t_001_pgbench_with_server_main_data/001_pgbench_log_2
> --sampling-rate=0.5
> ok 345 - pgbench logs status (got 0 vs expected 0)
> ok 346 - pgbench logs stdout /(?^:select only)/
> ok 347 - pgbench logs stdout /(?^:processed: 100/100)/
> ok 348 - pgbench logs stderr /(?^:^$)/
> not ok 349 - number of log files
>
>> If so, we might be able to fix it by converting backslashes to
>> forward slashes before applying quotemeta().
>>
>> It's also possible that on Windows, "glob" handles backslashes
>> differently than it does elsewhere, which would be harder to fix
>> --- that would bring back my original fear that the appropriate
>> quoting rules are different for "glob" than for regexes.
>
> I'm afraid it could indeed be due to platform-specific behavior, so
> testing on the target machine to understand the actual behavior would
> help.
>
> I just tested the current version on my laptop within a directory
> containing spaces and other misc chars: Pgbench TAP tests are
> currently broken in this context on master, and this may be used to a
> collection of issues, not just one, eg pgbench function splits
> parameters on spaces which breaks if there are spaces in bdir.
>
> I'm going to investigate, possibly over next week-end, so please be
> patient.
>
> About windows-specific issues, from File::Glob man page:
>
> """ On DOSISH systems, backslash is a valid directory separator
> character. In this case, use of backslash as a quoting character (via
> GLOB_QUOTE) interferes with the use of backslash as a directory
> separator. ... """
>
> It seems to suggest that quotemeta backslash-on-nearly-everything
> approach is not appropriate.
>
File globbing is not the same thing as regex matching. For example, '.'
is a literal character in a glob pattern but a metacharacter in a regex,
while ' ' is a literal character in a regex but a globbing metacharacter
(but see below), and '*' is a metacharacter in both but has different
meanings. quotemeta is intended for regexes but being used here on an
expression used for globbing.
Perhaps it would be OK we changed back the glob line to use $prefix
instead of $qprefix, but kept the use of $qprefix in the later regex.
To deal with the issue of spaces in file names (not an issue eher ob
bowerbird), we should consider adding this:
use File::Glob ':bsd_glob';
This removes the globbing metcharacter nature of the space character,
although it might have other effects that cause pain. See `perldoc
File::Glob`
cheers
andrew
--
Andrew Dunstan https://www.2ndQuadrant.com
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services
^ permalink raw reply [nested|flat] 22+ messages in thread
* Re: [PATCH] pgbench tap tests fail if the path contains a perl special character
2019-01-18 17:15 [PATCH] pgbench tap tests fail if the path contains a perl special character Raúl Marín Rodríguez <rmrodriguez@carto.com>
2019-01-18 17:35 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Tom Lane <tgl@sss.pgh.pa.us>
2019-01-18 17:54 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Raúl Marín Rodríguez <rmrodriguez@carto.com>
2019-01-18 19:27 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Alvaro Herrera <alvherre@2ndquadrant.com>
2019-01-18 20:02 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Tom Lane <tgl@sss.pgh.pa.us>
2019-01-18 20:26 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Tom Lane <tgl@sss.pgh.pa.us>
2019-01-18 20:56 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Raúl Marín Rodríguez <rmrodriguez@carto.com>
2019-01-21 01:51 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Tom Lane <tgl@sss.pgh.pa.us>
2019-01-21 09:50 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Fabien COELHO <coelho@cri.ensmp.fr>
2019-01-21 15:40 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Andrew Dunstan <andrew.dunstan@2ndquadrant.com>
@ 2019-01-21 16:10 ` Tom Lane <tgl@sss.pgh.pa.us>
2019-01-21 16:18 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Fabien COELHO <coelho@cri.ensmp.fr>
1 sibling, 1 reply; 22+ messages in thread
From: Tom Lane @ 2019-01-21 16:10 UTC (permalink / raw)
To: Andrew Dunstan <andrew.dunstan@2ndquadrant.com>; +Cc: Fabien COELHO <coelho@cri.ensmp.fr>; Raúl Marín Rodríguez <rmrodriguez@carto.com>; Alvaro Herrera <alvherre@2ndquadrant.com>; pgsql-hackers
Andrew Dunstan <andrew.dunstan@2ndquadrant.com> writes:
> Perhaps it would be OK we changed back the glob line to use $prefix
> instead of $qprefix, but kept the use of $qprefix in the later regex.
Well, glob does have some metacharacters, so not doing any quoting
at all still leaves us with failure modes.
I did a little bit of googling on this subject last night, and it
seems that at least some people believe that the answer is to not
use glob, period, but read the directory for yourself. Maybe there's
a better way, but the breakage around backslashes doesn't really
leave me hopeful.
As a short-term move to un-break the buildfarm, I'm just going to
revert that patch altogether. We can reapply it once we've
figured out how to do the glob part correctly.
regards, tom lane
^ permalink raw reply [nested|flat] 22+ messages in thread
* Re: [PATCH] pgbench tap tests fail if the path contains a perl special character
2019-01-18 17:15 [PATCH] pgbench tap tests fail if the path contains a perl special character Raúl Marín Rodríguez <rmrodriguez@carto.com>
2019-01-18 17:35 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Tom Lane <tgl@sss.pgh.pa.us>
2019-01-18 17:54 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Raúl Marín Rodríguez <rmrodriguez@carto.com>
2019-01-18 19:27 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Alvaro Herrera <alvherre@2ndquadrant.com>
2019-01-18 20:02 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Tom Lane <tgl@sss.pgh.pa.us>
2019-01-18 20:26 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Tom Lane <tgl@sss.pgh.pa.us>
2019-01-18 20:56 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Raúl Marín Rodríguez <rmrodriguez@carto.com>
2019-01-21 01:51 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Tom Lane <tgl@sss.pgh.pa.us>
2019-01-21 09:50 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Fabien COELHO <coelho@cri.ensmp.fr>
2019-01-21 15:40 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Andrew Dunstan <andrew.dunstan@2ndquadrant.com>
2019-01-21 16:10 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Tom Lane <tgl@sss.pgh.pa.us>
@ 2019-01-21 16:18 ` Fabien COELHO <coelho@cri.ensmp.fr>
2019-01-21 16:32 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Andrew Dunstan <andrew.dunstan@2ndquadrant.com>
2019-01-22 08:58 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Fabien COELHO <coelho@cri.ensmp.fr>
0 siblings, 2 replies; 22+ messages in thread
From: Fabien COELHO @ 2019-01-21 16:18 UTC (permalink / raw)
To: Tom Lane <tgl@sss.pgh.pa.us>; +Cc: Andrew Dunstan <andrew.dunstan@2ndquadrant.com>; Raúl Marín Rodríguez <rmrodriguez@carto.com>; Alvaro Herrera <alvherre@2ndquadrant.com>; pgsql-hackers
Hello Tom,
> Well, glob does have some metacharacters, so not doing any quoting
> at all still leaves us with failure modes.
>
> I did a little bit of googling on this subject last night, and it
> seems that at least some people believe that the answer is to not
> use glob, period, but read the directory for yourself.
Yep, would work, it can then be filtered with standard regexpr, although
it will take a few lines (opendir/grep on readdir/closedir).
> Maybe there's a better way, but the breakage around backslashes doesn't
> really leave me hopeful.
Indeed. I was thinking of defining my own quote function, but without
being convinced that it is such a good idea.
> As a short-term move to un-break the buildfarm, I'm just going to
> revert that patch altogether.
I was considering to suggest that, so I'm ok with that.
> We can reapply it once we've figured out how to do the glob part
> correctly.
Yep. No need to spend time much on that, I'll have a look at it, although
not right now, allow me a few days.
--
Fabien.
^ permalink raw reply [nested|flat] 22+ messages in thread
* Re: [PATCH] pgbench tap tests fail if the path contains a perl special character
2019-01-18 17:15 [PATCH] pgbench tap tests fail if the path contains a perl special character Raúl Marín Rodríguez <rmrodriguez@carto.com>
2019-01-18 17:35 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Tom Lane <tgl@sss.pgh.pa.us>
2019-01-18 17:54 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Raúl Marín Rodríguez <rmrodriguez@carto.com>
2019-01-18 19:27 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Alvaro Herrera <alvherre@2ndquadrant.com>
2019-01-18 20:02 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Tom Lane <tgl@sss.pgh.pa.us>
2019-01-18 20:26 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Tom Lane <tgl@sss.pgh.pa.us>
2019-01-18 20:56 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Raúl Marín Rodríguez <rmrodriguez@carto.com>
2019-01-21 01:51 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Tom Lane <tgl@sss.pgh.pa.us>
2019-01-21 09:50 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Fabien COELHO <coelho@cri.ensmp.fr>
2019-01-21 15:40 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Andrew Dunstan <andrew.dunstan@2ndquadrant.com>
2019-01-21 16:10 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Tom Lane <tgl@sss.pgh.pa.us>
2019-01-21 16:18 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Fabien COELHO <coelho@cri.ensmp.fr>
@ 2019-01-21 16:32 ` Andrew Dunstan <andrew.dunstan@2ndquadrant.com>
1 sibling, 0 replies; 22+ messages in thread
From: Andrew Dunstan @ 2019-01-21 16:32 UTC (permalink / raw)
To: Fabien COELHO <coelho@cri.ensmp.fr>; Tom Lane <tgl@sss.pgh.pa.us>; +Cc: Raúl Marín Rodríguez <rmrodriguez@carto.com>; Alvaro Herrera <alvherre@2ndquadrant.com>; pgsql-hackers
On 1/21/19 11:18 AM, Fabien COELHO wrote:
>
> Hello Tom,
>
>> Well, glob does have some metacharacters, so not doing any quoting
>> at all still leaves us with failure modes.
>>
>> I did a little bit of googling on this subject last night, and it
>> seems that at least some people believe that the answer is to not
>> use glob, period, but read the directory for yourself.
>
> Yep, would work, it can then be filtered with standard regexpr,
> although it will take a few lines (opendir/grep on readdir/closedir).
Sure, probably the best solution. Given that Perl has
opendir/readdir/closedir, it should be only a handful of lines.
cheers
andrew
--
Andrew Dunstan https://www.2ndQuadrant.com
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services
^ permalink raw reply [nested|flat] 22+ messages in thread
* Re: [PATCH] pgbench tap tests fail if the path contains a perl special character
2019-01-18 17:15 [PATCH] pgbench tap tests fail if the path contains a perl special character Raúl Marín Rodríguez <rmrodriguez@carto.com>
2019-01-18 17:35 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Tom Lane <tgl@sss.pgh.pa.us>
2019-01-18 17:54 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Raúl Marín Rodríguez <rmrodriguez@carto.com>
2019-01-18 19:27 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Alvaro Herrera <alvherre@2ndquadrant.com>
2019-01-18 20:02 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Tom Lane <tgl@sss.pgh.pa.us>
2019-01-18 20:26 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Tom Lane <tgl@sss.pgh.pa.us>
2019-01-18 20:56 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Raúl Marín Rodríguez <rmrodriguez@carto.com>
2019-01-21 01:51 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Tom Lane <tgl@sss.pgh.pa.us>
2019-01-21 09:50 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Fabien COELHO <coelho@cri.ensmp.fr>
2019-01-21 15:40 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Andrew Dunstan <andrew.dunstan@2ndquadrant.com>
2019-01-21 16:10 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Tom Lane <tgl@sss.pgh.pa.us>
2019-01-21 16:18 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Fabien COELHO <coelho@cri.ensmp.fr>
@ 2019-01-22 08:58 ` Fabien COELHO <coelho@cri.ensmp.fr>
2019-01-22 10:20 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Raúl Marín Rodríguez <rmrodriguez@carto.com>
1 sibling, 1 reply; 22+ messages in thread
From: Fabien COELHO @ 2019-01-22 08:58 UTC (permalink / raw)
To: Tom Lane <tgl@sss.pgh.pa.us>; +Cc: Andrew Dunstan <andrew.dunstan@2ndquadrant.com>; Raúl Marín Rodríguez <rmrodriguez@carto.com>; Alvaro Herrera <alvherre@2ndquadrant.com>; pgsql-hackers
Hello Tom,
> I did a little bit of googling on this subject last night, and it
> seems that at least some people believe that the answer is to not
> use glob, period, but read the directory for yourself.
>
> As a short-term move to un-break the buildfarm, I'm just going to
> revert that patch altogether.
>
> We can reapply it once we've figured out how to do the glob part correctly.
Here is a proposal patch which:
- works around pgbench command splitting on spaces,
if postgres sources are in a strangely named directory…
I tested within a directory named "pg .* dir".
- works aroung "glob" lack of portability by reading the directory
directly and filtering with a standard re (see list_files).
- adds a few comments
- removes a spurious i option on an empty re
--
Fabien.
Attachments:
[text/x-diff] pgbench-tap-glob-fix-1.patch (3.4K, ../../alpine.DEB.2.21.1901220945370.15365@lancre/2-pgbench-tap-glob-fix-1.patch)
download | inline diff:
diff --git a/src/bin/pgbench/t/001_pgbench_with_server.pl b/src/bin/pgbench/t/001_pgbench_with_server.pl
index c87748086a..53a6d0c926 100644
--- a/src/bin/pgbench/t/001_pgbench_with_server.pl
+++ b/src/bin/pgbench/t/001_pgbench_with_server.pl
@@ -10,12 +10,19 @@ my $node = get_new_node('main');
$node->init;
$node->start;
-# invoke pgbench
+# invoke pgbench, with parameters:
+# $opts: options as a string to be split on spaces
+# $stat: expected exit status
+# $out: reference to a regexp list that must match stdout
+# $err: reference to a regexp list that must match stderr
+# $name: name of test for error messages
+# $files: reference to filename/contents dictionnary
+# @args: further raw options or arguments
sub pgbench
{
local $Test::Builder::Level = $Test::Builder::Level + 1;
- my ($opts, $stat, $out, $err, $name, $files) = @_;
+ my ($opts, $stat, $out, $err, $name, $files, @args) = @_;
my @cmd = ('pgbench', split /\s+/, $opts);
my @filenames = ();
if (defined $files)
@@ -40,6 +47,9 @@ sub pgbench
append_to_file($filename, $$files{$fn});
}
}
+
+ push @cmd, @args;
+
$node->command_checks_all(\@cmd, $stat, $out, $err, $name);
# cleanup?
@@ -868,20 +878,32 @@ pgbench(
qr{type: .*/001_pgbench_sleep},
qr{above the 1.0 ms latency limit: [01]/}
],
- [qr{^$}i],
+ [qr{^$}],
'pgbench late throttling',
{ '001_pgbench_sleep' => q{\sleep 2ms} });
+# return a list of files from directory $dir matching regexpr $re
+# this works around glob portability and escaping issues
+sub list_files
+{
+ my ($dir, $re) = @_;
+ opendir my $dh, $dir or die "cannot opendir $dir: $!";
+ my @files = grep /$re/, readdir $dh;
+ closedir $dh or die "cannot closedir $dir: $!";
+ return map { $dir . '/' . $_ } @files;
+}
+
# check log contents and cleanup
sub check_pgbench_logs
{
local $Test::Builder::Level = $Test::Builder::Level + 1;
- my ($prefix, $nb, $min, $max, $re) = @_;
+ my ($dir, $prefix, $nb, $min, $max, $re) = @_;
- my @logs = glob "$prefix.*";
+ # $prefix is simple enough, thus does not need escaping
+ my @logs = list_files($dir, qr{^$prefix\..*$});
ok(@logs == $nb, "number of log files");
- ok(grep(/^$prefix\.\d+(\.\d+)?$/, @logs) == $nb, "file name format");
+ ok(grep(/\/$prefix\.\d+(\.\d+)?$/, @logs) == $nb, "file name format");
my $log_number = 0;
for my $log (sort @logs)
@@ -905,22 +927,25 @@ my $bdir = $node->basedir;
# with sampling rate
pgbench(
- "-n -S -t 50 -c 2 --log --log-prefix=$bdir/001_pgbench_log_2 --sampling-rate=0.5",
+ "-n -S -t 50 -c 2 --log --sampling-rate=0.5",
0,
[ qr{select only}, qr{processed: 100/100} ],
- [qr{^$}],
- 'pgbench logs');
+ [ qr{^$} ],
+ 'pgbench logs',
+ undef,
+ "--log-prefix=$bdir/001_pgbench_log_2");
-check_pgbench_logs("$bdir/001_pgbench_log_2", 1, 8, 92,
+check_pgbench_logs($bdir, '001_pgbench_log_2', 1, 8, 92,
qr{^0 \d{1,2} \d+ \d \d+ \d+$});
# check log file in some detail
pgbench(
- "-n -b se -t 10 -l --log-prefix=$bdir/001_pgbench_log_3",
- 0, [ qr{select only}, qr{processed: 10/10} ],
- [qr{^$}], 'pgbench logs contents');
+ "-n -b se -t 10 -l",
+ 0, [ qr{select only}, qr{processed: 10/10} ], [ qr{^$} ],
+ 'pgbench logs contents', undef,
+ "--log-prefix=$bdir/001_pgbench_log_3");
-check_pgbench_logs("$bdir/001_pgbench_log_3", 1, 10, 10,
+check_pgbench_logs($bdir, '001_pgbench_log_3', 1, 10, 10,
qr{^\d \d{1,2} \d+ \d \d+ \d+$});
# done
^ permalink raw reply [nested|flat] 22+ messages in thread
* Re: [PATCH] pgbench tap tests fail if the path contains a perl special character
2019-01-18 17:15 [PATCH] pgbench tap tests fail if the path contains a perl special character Raúl Marín Rodríguez <rmrodriguez@carto.com>
2019-01-18 17:35 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Tom Lane <tgl@sss.pgh.pa.us>
2019-01-18 17:54 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Raúl Marín Rodríguez <rmrodriguez@carto.com>
2019-01-18 19:27 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Alvaro Herrera <alvherre@2ndquadrant.com>
2019-01-18 20:02 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Tom Lane <tgl@sss.pgh.pa.us>
2019-01-18 20:26 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Tom Lane <tgl@sss.pgh.pa.us>
2019-01-18 20:56 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Raúl Marín Rodríguez <rmrodriguez@carto.com>
2019-01-21 01:51 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Tom Lane <tgl@sss.pgh.pa.us>
2019-01-21 09:50 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Fabien COELHO <coelho@cri.ensmp.fr>
2019-01-21 15:40 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Andrew Dunstan <andrew.dunstan@2ndquadrant.com>
2019-01-21 16:10 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Tom Lane <tgl@sss.pgh.pa.us>
2019-01-21 16:18 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Fabien COELHO <coelho@cri.ensmp.fr>
2019-01-22 08:58 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Fabien COELHO <coelho@cri.ensmp.fr>
@ 2019-01-22 10:20 ` Raúl Marín Rodríguez <rmrodriguez@carto.com>
2019-01-22 10:31 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Fabien COELHO <coelho@cri.ensmp.fr>
0 siblings, 1 reply; 22+ messages in thread
From: Raúl Marín Rodríguez @ 2019-01-22 10:20 UTC (permalink / raw)
To: Fabien COELHO <coelho@cri.ensmp.fr>; +Cc: Tom Lane <tgl@sss.pgh.pa.us>; Andrew Dunstan <andrew.dunstan@2ndquadrant.com>; Alvaro Herrera <alvherre@2ndquadrant.com>; pgsql-hackers
Hi,
> - works around pgbench command splitting on spaces,
> if postgres sources are in a strangely named directory…
> I tested within a directory named "pg .* dir".
I've also tested it with the initial case (including a +) and it works.
--
Raúl Marín Rodríguez
carto.com
^ permalink raw reply [nested|flat] 22+ messages in thread
* Re: [PATCH] pgbench tap tests fail if the path contains a perl special character
2019-01-18 17:15 [PATCH] pgbench tap tests fail if the path contains a perl special character Raúl Marín Rodríguez <rmrodriguez@carto.com>
2019-01-18 17:35 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Tom Lane <tgl@sss.pgh.pa.us>
2019-01-18 17:54 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Raúl Marín Rodríguez <rmrodriguez@carto.com>
2019-01-18 19:27 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Alvaro Herrera <alvherre@2ndquadrant.com>
2019-01-18 20:02 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Tom Lane <tgl@sss.pgh.pa.us>
2019-01-18 20:26 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Tom Lane <tgl@sss.pgh.pa.us>
2019-01-18 20:56 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Raúl Marín Rodríguez <rmrodriguez@carto.com>
2019-01-21 01:51 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Tom Lane <tgl@sss.pgh.pa.us>
2019-01-21 09:50 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Fabien COELHO <coelho@cri.ensmp.fr>
2019-01-21 15:40 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Andrew Dunstan <andrew.dunstan@2ndquadrant.com>
2019-01-21 16:10 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Tom Lane <tgl@sss.pgh.pa.us>
2019-01-21 16:18 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Fabien COELHO <coelho@cri.ensmp.fr>
2019-01-22 08:58 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Fabien COELHO <coelho@cri.ensmp.fr>
2019-01-22 10:20 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Raúl Marín Rodríguez <rmrodriguez@carto.com>
@ 2019-01-22 10:31 ` Fabien COELHO <coelho@cri.ensmp.fr>
2019-02-08 12:33 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Raúl Marín Rodríguez <rmrodriguez@carto.com>
0 siblings, 1 reply; 22+ messages in thread
From: Fabien COELHO @ 2019-01-22 10:31 UTC (permalink / raw)
To: Raúl Marín Rodríguez <rmrodriguez@carto.com>; +Cc: Tom Lane <tgl@sss.pgh.pa.us>; Andrew Dunstan <andrew.dunstan@2ndquadrant.com>; Alvaro Herrera <alvherre@2ndquadrant.com>; pgsql-hackers
>> - works around pgbench command splitting on spaces,
>> if postgres sources are in a strangely named directory…
>> I tested within a directory named "pg .* dir".
>
> I've also tested it with the initial case (including a +) and it works.
Good, thanks for checking!
--
Fabien.
^ permalink raw reply [nested|flat] 22+ messages in thread
* Re: [PATCH] pgbench tap tests fail if the path contains a perl special character
2019-01-18 17:15 [PATCH] pgbench tap tests fail if the path contains a perl special character Raúl Marín Rodríguez <rmrodriguez@carto.com>
2019-01-18 17:35 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Tom Lane <tgl@sss.pgh.pa.us>
2019-01-18 17:54 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Raúl Marín Rodríguez <rmrodriguez@carto.com>
2019-01-18 19:27 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Alvaro Herrera <alvherre@2ndquadrant.com>
2019-01-18 20:02 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Tom Lane <tgl@sss.pgh.pa.us>
2019-01-18 20:26 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Tom Lane <tgl@sss.pgh.pa.us>
2019-01-18 20:56 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Raúl Marín Rodríguez <rmrodriguez@carto.com>
2019-01-21 01:51 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Tom Lane <tgl@sss.pgh.pa.us>
2019-01-21 09:50 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Fabien COELHO <coelho@cri.ensmp.fr>
2019-01-21 15:40 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Andrew Dunstan <andrew.dunstan@2ndquadrant.com>
2019-01-21 16:10 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Tom Lane <tgl@sss.pgh.pa.us>
2019-01-21 16:18 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Fabien COELHO <coelho@cri.ensmp.fr>
2019-01-22 08:58 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Fabien COELHO <coelho@cri.ensmp.fr>
2019-01-22 10:20 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Raúl Marín Rodríguez <rmrodriguez@carto.com>
2019-01-22 10:31 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Fabien COELHO <coelho@cri.ensmp.fr>
@ 2019-02-08 12:33 ` Raúl Marín Rodríguez <rmrodriguez@carto.com>
2019-02-08 13:18 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Daniel Gustafsson <daniel@yesql.se>
0 siblings, 1 reply; 22+ messages in thread
From: Raúl Marín Rodríguez @ 2019-02-08 12:33 UTC (permalink / raw)
To: Fabien COELHO <coelho@cri.ensmp.fr>; +Cc: Tom Lane <tgl@sss.pgh.pa.us>; Andrew Dunstan <andrew.dunstan@2ndquadrant.com>; Alvaro Herrera <alvherre@2ndquadrant.com>; pgsql-hackers
Hi,
Any way I can help to get this commited? Should we add this to the commitfest?
Regards,
--
Raúl Marín Rodríguez
carto.com
^ permalink raw reply [nested|flat] 22+ messages in thread
* Re: [PATCH] pgbench tap tests fail if the path contains a perl special character
2019-01-18 17:15 [PATCH] pgbench tap tests fail if the path contains a perl special character Raúl Marín Rodríguez <rmrodriguez@carto.com>
2019-01-18 17:35 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Tom Lane <tgl@sss.pgh.pa.us>
2019-01-18 17:54 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Raúl Marín Rodríguez <rmrodriguez@carto.com>
2019-01-18 19:27 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Alvaro Herrera <alvherre@2ndquadrant.com>
2019-01-18 20:02 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Tom Lane <tgl@sss.pgh.pa.us>
2019-01-18 20:26 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Tom Lane <tgl@sss.pgh.pa.us>
2019-01-18 20:56 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Raúl Marín Rodríguez <rmrodriguez@carto.com>
2019-01-21 01:51 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Tom Lane <tgl@sss.pgh.pa.us>
2019-01-21 09:50 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Fabien COELHO <coelho@cri.ensmp.fr>
2019-01-21 15:40 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Andrew Dunstan <andrew.dunstan@2ndquadrant.com>
2019-01-21 16:10 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Tom Lane <tgl@sss.pgh.pa.us>
2019-01-21 16:18 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Fabien COELHO <coelho@cri.ensmp.fr>
2019-01-22 08:58 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Fabien COELHO <coelho@cri.ensmp.fr>
2019-01-22 10:20 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Raúl Marín Rodríguez <rmrodriguez@carto.com>
2019-01-22 10:31 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Fabien COELHO <coelho@cri.ensmp.fr>
2019-02-08 12:33 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Raúl Marín Rodríguez <rmrodriguez@carto.com>
@ 2019-02-08 13:18 ` Daniel Gustafsson <daniel@yesql.se>
2019-02-08 13:30 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Raúl Marín <git@rmr.ninja>
0 siblings, 1 reply; 22+ messages in thread
From: Daniel Gustafsson @ 2019-02-08 13:18 UTC (permalink / raw)
To: Raúl Marín Rodríguez <rmrodriguez@carto.com>; +Cc: Fabien COELHO <coelho@cri.ensmp.fr>; Tom Lane <tgl@sss.pgh.pa.us>; Andrew Dunstan <andrew.dunstan@2ndquadrant.com>; Alvaro Herrera <alvherre@2ndquadrant.com>; pgsql-hackers
> On 8 Feb 2019, at 13:33, Raúl Marín Rodríguez <rmrodriguez@carto.com> wrote:
> Any way I can help to get this commited? Should we add this to the commitfest?
Please do, it’s a good way to ensure that the patch is tracked and not
forgotten about.
cheers ./daniel
^ permalink raw reply [nested|flat] 22+ messages in thread
* Re: [PATCH] pgbench tap tests fail if the path contains a perl special character
2019-01-18 17:15 [PATCH] pgbench tap tests fail if the path contains a perl special character Raúl Marín Rodríguez <rmrodriguez@carto.com>
2019-01-18 17:35 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Tom Lane <tgl@sss.pgh.pa.us>
2019-01-18 17:54 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Raúl Marín Rodríguez <rmrodriguez@carto.com>
2019-01-18 19:27 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Alvaro Herrera <alvherre@2ndquadrant.com>
2019-01-18 20:02 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Tom Lane <tgl@sss.pgh.pa.us>
2019-01-18 20:26 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Tom Lane <tgl@sss.pgh.pa.us>
2019-01-18 20:56 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Raúl Marín Rodríguez <rmrodriguez@carto.com>
2019-01-21 01:51 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Tom Lane <tgl@sss.pgh.pa.us>
2019-01-21 09:50 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Fabien COELHO <coelho@cri.ensmp.fr>
2019-01-21 15:40 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Andrew Dunstan <andrew.dunstan@2ndquadrant.com>
2019-01-21 16:10 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Tom Lane <tgl@sss.pgh.pa.us>
2019-01-21 16:18 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Fabien COELHO <coelho@cri.ensmp.fr>
2019-01-22 08:58 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Fabien COELHO <coelho@cri.ensmp.fr>
2019-01-22 10:20 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Raúl Marín Rodríguez <rmrodriguez@carto.com>
2019-01-22 10:31 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Fabien COELHO <coelho@cri.ensmp.fr>
2019-02-08 12:33 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Raúl Marín Rodríguez <rmrodriguez@carto.com>
2019-02-08 13:18 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Daniel Gustafsson <daniel@yesql.se>
@ 2019-02-08 13:30 ` Raúl Marín <git@rmr.ninja>
2019-03-05 08:10 ` Re: Re: [PATCH] pgbench tap tests fail if the path contains a perl special character David Steele <david@pgmasters.net>
0 siblings, 1 reply; 22+ messages in thread
From: Raúl Marín @ 2019-02-08 13:30 UTC (permalink / raw)
To: pgsql-hackers@lists.postgresql.org; +Cc: Fabien Coelho <postgresql.org@coelho.net>
The following review has been posted through the commitfest application:
make installcheck-world: tested, failed
Implements feature: not tested
Spec compliant: not tested
Documentation: not tested
Tested `pgbench-tap-glob-fix-1.patch` with a "+" in the path and the tap tests now pass.
^ permalink raw reply [nested|flat] 22+ messages in thread
* Re: Re: [PATCH] pgbench tap tests fail if the path contains a perl special character
2019-01-18 17:15 [PATCH] pgbench tap tests fail if the path contains a perl special character Raúl Marín Rodríguez <rmrodriguez@carto.com>
2019-01-18 17:35 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Tom Lane <tgl@sss.pgh.pa.us>
2019-01-18 17:54 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Raúl Marín Rodríguez <rmrodriguez@carto.com>
2019-01-18 19:27 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Alvaro Herrera <alvherre@2ndquadrant.com>
2019-01-18 20:02 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Tom Lane <tgl@sss.pgh.pa.us>
2019-01-18 20:26 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Tom Lane <tgl@sss.pgh.pa.us>
2019-01-18 20:56 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Raúl Marín Rodríguez <rmrodriguez@carto.com>
2019-01-21 01:51 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Tom Lane <tgl@sss.pgh.pa.us>
2019-01-21 09:50 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Fabien COELHO <coelho@cri.ensmp.fr>
2019-01-21 15:40 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Andrew Dunstan <andrew.dunstan@2ndquadrant.com>
2019-01-21 16:10 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Tom Lane <tgl@sss.pgh.pa.us>
2019-01-21 16:18 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Fabien COELHO <coelho@cri.ensmp.fr>
2019-01-22 08:58 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Fabien COELHO <coelho@cri.ensmp.fr>
2019-01-22 10:20 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Raúl Marín Rodríguez <rmrodriguez@carto.com>
2019-01-22 10:31 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Fabien COELHO <coelho@cri.ensmp.fr>
2019-02-08 12:33 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Raúl Marín Rodríguez <rmrodriguez@carto.com>
2019-02-08 13:18 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Daniel Gustafsson <daniel@yesql.se>
2019-02-08 13:30 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Raúl Marín <git@rmr.ninja>
@ 2019-03-05 08:10 ` David Steele <david@pgmasters.net>
2019-03-05 11:00 ` Re: Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Raúl Marín Rodríguez <rmrodriguez@carto.com>
0 siblings, 1 reply; 22+ messages in thread
From: David Steele @ 2019-03-05 08:10 UTC (permalink / raw)
To: Raúl Marín <git@rmr.ninja>; pgsql-hackers@lists.postgresql.org; +Cc: Fabien Coelho <postgresql.org@coelho.net>
On 2/8/19 3:30 PM, Raúl Marín wrote:
> The following review has been posted through the commitfest application:
> make installcheck-world: tested, failed
> Implements feature: not tested
> Spec compliant: not tested
> Documentation: not tested
>
> Tested `pgbench-tap-glob-fix-1.patch` with a "+" in the path and the tap tests now pass.
Is this now ready for committer or is there anything else Fabien needs
to do?
Marking the patch as either Ready for Committer or Waiting on Author
will help move things along.
Regards,
--
-David
david@pgmasters.net
^ permalink raw reply [nested|flat] 22+ messages in thread
* Re: Re: [PATCH] pgbench tap tests fail if the path contains a perl special character
2019-01-18 17:15 [PATCH] pgbench tap tests fail if the path contains a perl special character Raúl Marín Rodríguez <rmrodriguez@carto.com>
2019-01-18 17:35 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Tom Lane <tgl@sss.pgh.pa.us>
2019-01-18 17:54 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Raúl Marín Rodríguez <rmrodriguez@carto.com>
2019-01-18 19:27 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Alvaro Herrera <alvherre@2ndquadrant.com>
2019-01-18 20:02 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Tom Lane <tgl@sss.pgh.pa.us>
2019-01-18 20:26 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Tom Lane <tgl@sss.pgh.pa.us>
2019-01-18 20:56 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Raúl Marín Rodríguez <rmrodriguez@carto.com>
2019-01-21 01:51 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Tom Lane <tgl@sss.pgh.pa.us>
2019-01-21 09:50 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Fabien COELHO <coelho@cri.ensmp.fr>
2019-01-21 15:40 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Andrew Dunstan <andrew.dunstan@2ndquadrant.com>
2019-01-21 16:10 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Tom Lane <tgl@sss.pgh.pa.us>
2019-01-21 16:18 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Fabien COELHO <coelho@cri.ensmp.fr>
2019-01-22 08:58 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Fabien COELHO <coelho@cri.ensmp.fr>
2019-01-22 10:20 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Raúl Marín Rodríguez <rmrodriguez@carto.com>
2019-01-22 10:31 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Fabien COELHO <coelho@cri.ensmp.fr>
2019-02-08 12:33 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Raúl Marín Rodríguez <rmrodriguez@carto.com>
2019-02-08 13:18 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Daniel Gustafsson <daniel@yesql.se>
2019-02-08 13:30 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Raúl Marín <git@rmr.ninja>
2019-03-05 08:10 ` Re: Re: [PATCH] pgbench tap tests fail if the path contains a perl special character David Steele <david@pgmasters.net>
@ 2019-03-05 11:00 ` Raúl Marín Rodríguez <rmrodriguez@carto.com>
0 siblings, 0 replies; 22+ messages in thread
From: Raúl Marín Rodríguez @ 2019-03-05 11:00 UTC (permalink / raw)
To: David Steele <david@pgmasters.net>; +Cc: Raúl Marín <git@rmr.ninja>; pgsql-hackers@lists.postgresql.org, Fabien Coelho <postgresql.org@coelho.net>
> Marking the patch as either Ready for Committer or Waiting on Author
will help move things along.
Thanks for the heads-up, I've retested it again and it applies cleanly on
top of master and addresses the issue.
I've moved it to `Ready for Committer`.
--
Raúl Marín Rodríguez
carto.com
^ permalink raw reply [nested|flat] 22+ messages in thread
* Re: [PATCH] pgbench tap tests fail if the path contains a perl special character
2019-01-18 17:15 [PATCH] pgbench tap tests fail if the path contains a perl special character Raúl Marín Rodríguez <rmrodriguez@carto.com>
2019-01-18 17:35 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Tom Lane <tgl@sss.pgh.pa.us>
2019-01-18 17:54 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Raúl Marín Rodríguez <rmrodriguez@carto.com>
2019-01-18 19:27 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Alvaro Herrera <alvherre@2ndquadrant.com>
2019-01-18 20:02 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Tom Lane <tgl@sss.pgh.pa.us>
2019-01-18 20:26 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Tom Lane <tgl@sss.pgh.pa.us>
2019-01-18 20:56 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Raúl Marín Rodríguez <rmrodriguez@carto.com>
2019-01-21 01:51 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Tom Lane <tgl@sss.pgh.pa.us>
2019-01-21 09:50 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Fabien COELHO <coelho@cri.ensmp.fr>
2019-01-21 15:40 ` Re: [PATCH] pgbench tap tests fail if the path contains a perl special character Andrew Dunstan <andrew.dunstan@2ndquadrant.com>
@ 2019-01-21 16:11 ` Fabien COELHO <coelho@cri.ensmp.fr>
1 sibling, 0 replies; 22+ messages in thread
From: Fabien COELHO @ 2019-01-21 16:11 UTC (permalink / raw)
To: Andrew Dunstan <andrew.dunstan@2ndquadrant.com>; +Cc: Tom Lane <tgl@sss.pgh.pa.us>; Raúl Marín Rodríguez <rmrodriguez@carto.com>; Alvaro Herrera <alvherre@2ndquadrant.com>; pgsql-hackers
Hello Andrew,
>> About windows-specific issues, from File::Glob man page:
>>
>> """ On DOSISH systems, backslash is a valid directory separator
>> character. In this case, use of backslash as a quoting character (via
>> GLOB_QUOTE) interferes with the use of backslash as a directory
>> separator. ... """
>>
>> It seems to suggest that quotemeta backslash-on-nearly-everything
>> approach is not appropriate.
>
> File globbing is not the same thing as regex matching. For example, '.'
> is a literal character in a glob pattern but a metacharacter in a regex,
> while ' ' is a literal character in a regex but a globbing metacharacter
> (but see below), and '*' is a metacharacter in both but has different
> meanings. quotemeta is intended for regexes but being used here on an
> expression used for globbing.
>
> Perhaps it would be OK we changed back the glob line to use $prefix
> instead of $qprefix, but kept the use of $qprefix in the later regex.
Yep, possibly. I'll have to test, though.
> To deal with the issue of spaces in file names (not an issue eher ob
> bowerbird), we should consider adding this:
>
> use File::Glob ':bsd_glob';
I was planning that so that the behavior is the same on all systems.
> This removes the globbing metcharacter nature of the space character,
> although it might have other effects that cause pain. See `perldoc
> File::Glob`
Yep. Thanks for doing my homework:-) I'll test starting with these
options.
--
Fabien.
^ permalink raw reply [nested|flat] 22+ messages in thread
end of thread, other threads:[~2019-03-05 11:00 UTC | newest]
Thread overview: 22+ messages (download: mbox mbox.gz follow: Atom feed)
-- links below jump to the message on this page --
2019-01-18 17:15 [PATCH] pgbench tap tests fail if the path contains a perl special character Raúl Marín Rodríguez <rmrodriguez@carto.com>
2019-01-18 17:35 ` Tom Lane <tgl@sss.pgh.pa.us>
2019-01-18 17:54 ` Raúl Marín Rodríguez <rmrodriguez@carto.com>
2019-01-18 19:27 ` Alvaro Herrera <alvherre@2ndquadrant.com>
2019-01-18 20:02 ` Tom Lane <tgl@sss.pgh.pa.us>
2019-01-18 20:26 ` Tom Lane <tgl@sss.pgh.pa.us>
2019-01-18 20:56 ` Raúl Marín Rodríguez <rmrodriguez@carto.com>
2019-01-21 01:51 ` Tom Lane <tgl@sss.pgh.pa.us>
2019-01-21 09:50 ` Fabien COELHO <coelho@cri.ensmp.fr>
2019-01-21 15:40 ` Andrew Dunstan <andrew.dunstan@2ndquadrant.com>
2019-01-21 16:10 ` Tom Lane <tgl@sss.pgh.pa.us>
2019-01-21 16:18 ` Fabien COELHO <coelho@cri.ensmp.fr>
2019-01-21 16:32 ` Andrew Dunstan <andrew.dunstan@2ndquadrant.com>
2019-01-22 08:58 ` Fabien COELHO <coelho@cri.ensmp.fr>
2019-01-22 10:20 ` Raúl Marín Rodríguez <rmrodriguez@carto.com>
2019-01-22 10:31 ` Fabien COELHO <coelho@cri.ensmp.fr>
2019-02-08 12:33 ` Raúl Marín Rodríguez <rmrodriguez@carto.com>
2019-02-08 13:18 ` Daniel Gustafsson <daniel@yesql.se>
2019-02-08 13:30 ` Raúl Marín <git@rmr.ninja>
2019-03-05 08:10 ` David Steele <david@pgmasters.net>
2019-03-05 11:00 ` Raúl Marín Rodríguez <rmrodriguez@carto.com>
2019-01-21 16:11 ` Fabien COELHO <coelho@cri.ensmp.fr>
This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox