Re: Parallel copy - Mailing list pgsql-hackers

From Daniel Westermann (DWE)
Subject Re: Parallel copy
Date
Msg-id ZR0P278MB010570202502E5F0F0E5FC3ED2140@ZR0P278MB0105.CHEP278.PROD.OUTLOOK.COM
Whole thread Raw
In response to Re: Parallel copy  (Heikki Linnakangas <hlinnaka@iki.fi>)
Responses Re: Parallel copy
List pgsql-hackers
On 27/10/2020 15:36, vignesh C wrote:
>> Attached v9 patches have the fixes for the above comments.

>I did some testing:

I did some testing as well and have a cosmetic remark:

postgres=# copy t1 from '/var/tmp/aa.txt' with (parallel 1000000000);
ERROR:  value 1000000000 out of bounds for option "parallel"
DETAIL:  Valid values are between "1" and "1024".
postgres=# copy t1 from '/var/tmp/aa.txt' with (parallel 100000000000);
ERROR:  parallel requires an integer value
postgres=#

Wouldn't it make more sense to only have one error message? The first one seems to be the better message.

Regards
Daniel


pgsql-hackers by date:

Previous
From: Heikki Linnakangas
Date:
Subject: Re: Parallel copy
Next
From: "osumi.takamichi@fujitsu.com"
Date:
Subject: RE: extension patch of CREATE OR REPLACE TRIGGER