| From: | Bharath Rupireddy <bharath(dot)rupireddyforpostgres(at)gmail(dot)com> |
|---|---|
| To: | vignesh C <vignesh21(at)gmail(dot)com> |
| Cc: | PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org> |
| Subject: | Re: Parallel copy |
| Date: | 2020-10-21 10:50:16 |
| Message-ID: | CALj2ACUu2794si=i05_0cQKW+fek8H0gQ_hUY1+m+gNkJiMANQ@mail.gmail.com |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On Wed, Oct 21, 2020 at 3:18 PM Bharath Rupireddy
<bharath(dot)rupireddyforpostgres(at)gmail(dot)com> wrote:
>
> 17. Remove extra lines after #define IsHeaderLine()
> (cstate->header_line && cstate->cur_lineno == 1) in copy.h
>
I missed one comment:
18. I think we need to treat the number of parallel workers as an
integer similar to the parallel option in vacuum.
postgres=# copy t1 from stdin with(parallel '1'); <<<<< - we
should not allow this.
Enter data to be copied followed by a newline.
postgres=# vacuum (parallel '1') t1;
ERROR: parallel requires an integer value
With Regards,
Bharath Rupireddy.
EnterpriseDB: http://www.enterprisedb.com
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Onder Kalaci | 2020-10-21 10:54:48 | Re: Combination of geqo and enable_partitionwise_join leads to crashes in the regression tests |
| Previous Message | Onder Kalaci | 2020-10-21 10:49:36 | Combination of geqo and enable_partitionwise_join leads to crashes in the regression tests |