From: | Josef Šimánek <josef(dot)simanek(at)gmail(dot)com> |
---|---|
To: | 0010203112132233 <boekewurm(at)gmail(dot)com> |
Cc: | Matthias van de Meent <boekewurm+postgres(at)gmail(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org>, Tomas Vondra <tomas(dot)vondra(at)enterprisedb(dot)com> |
Subject: | Re: Improvements and additions to COPY progress reporting |
Date: | 2021-02-09 11:53:44 |
Message-ID: | CAFp7QwotqR+aRP34nGiHRHZpEYDQQ1yZhzNDuE6MUQ=rLgNYbA@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
út 9. 2. 2021 v 12:51 odesílatel 0010203112132233 <boekewurm(at)gmail(dot)com> napsal:
>
> On Tue, 9 Feb 2021 at 09:32, Josef Šimánek <josef(dot)simanek(at)gmail(dot)com> wrote:
> >
> > po 8. 2. 2021 v 19:35 odesílatel Matthias van de Meent
> > <boekewurm+postgres(at)gmail(dot)com> napsal:
> > > Lastly, 0005 adds 'io_target' to the reported information, that is,
> > > FILE, PROGRAM, STDIO or CALLBACK. Although this can relatively easily
> > > be determined based on the commands in pg_stat_activity, it is
> > > reasonably something that a user would want to query on, as the
> > > origin/target of COPY has security and performance implications,
> > > whereas other options (e.g. format) are less interesting for clients
> > > that are not executing that specific COPY command.
> >
> > I took a little deeper look and I'm not sure if I understand FILE and
> > STDIO. I have finally tried to finalize some initial regress testing
> > of COPY command progress using triggers. I have attached the initial
> > patch applicable to your changes. As you can see COPY FROM STDIN is
> > reported as FILE. That's probably expected, but it is a little
> > confusing for me since STDIN and STDIO sound similar. What is the
> > purpose of STDIO? When is the COPY command reported with io_target of
> > STDIO?
>
> I checked for the type of the copy_src before it was correctly set,
> therefore only reporting FILE type, but this will be fixed shortly in
> v3.
OK, would you mind to integrate my regression test initial patch as
well in v3 or should I submit it later in a separate way?
> Matthias
From | Date | Subject | |
---|---|---|---|
Next Message | Amit Kapila | 2021-02-09 11:57:12 | Re: repeated decoding of prepared transactions |
Previous Message | 0010203112132233 | 2021-02-09 11:51:20 | Re: Improvements and additions to COPY progress reporting |